Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index



 Hello,

 I have a patch that is 32M big and I rather wonder when to commit it.

 The patch specifically is string rtl:: prefix removal. These changes 
regularly show up in gerrit patches for review, but I think we have somewhat 
better EasyHacks for new developers than rather mindless search&replace, and 
it's somewhat annoying to review such diffs as well. So instead I've done the 
change over the entire codebase where possible.

 The problem is, the change is so extensive that it touches pretty much 
everything ("9117 files changed, 92833 insertions(+), 95662 deletions(-)"). 
Given that we branched 4-0 a short time back, this will probably mean 
backporting will often need adjusting. So the options I see:

- wait until before 4-1 branch-off (or do not do such a big change at all) - 
this would mean there still would be these changes done in small bits by 
other people (including the gerrit review overhead)

- push now, only to master - this would mean more likely conflicts when 
backporting changes

- push now, including to 4-0 - I can successfully do a full build, including 
make check, and I've avoided public API modules (sal, salhelper, cppu, 
cppuhelper, odk - any others?), so this hopefully should be safe, but most of 
the changes have been done using sed, so there may be something that I've 
missed

 Does somebody have a preference? I consider the last one to be the best 
option.

-- 
 Lubos Lunak
 l.lunak@suse.cz

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.