[Libreoffice] merge issues ...
kendy at suse.cz
Tue May 31 06:35:07 PDT 2011
On 2011-05-31 at 14:30 +0200, Bjoern Michaelsen wrote:
> I just noted that commit
> got silently reverted by one of the recent merges. The commit
> itself is uncritical, but do we have a general problem here? It does
> not seem as if there was any manual intervention at all for this upon
I investigated, and it happened because the code introducing the
was pushed to both master and libreoffice-3-4, but the code removing
this line just to master. That correctly conflicted - and the person
doing the merge [me in this case - sorry for that] - picked the version
containing that line [either that I overlooked that, did not investigate
enough, or the tooling played tricks on me, and picked that
automatically - I don't remember].
> Silently dropping changes from master seems to be a serious issue to
All would be fine if either everything [the original code as well as the
fix] was pushed to both libreoffice-3-4 and master, or everything to
just one of these branches.
The conclusion is that whatever way anybody chooses (committing just to
release branch, or committing to master, and cherry-picking to the
release branch), the person has to be consistent in the choice when
there are subsequent fixes.
[And before you think "this wouldn't have happened if we never merged
the release branch back into master, because we would be always
cherry-picking" - I just have a counter-example of a fix that was
committed to master quite some time ago, but after the libreoffice-3-4
branch-off, and fell though the cracks - nobody noticed that it hasn't
been cherry-picked to libreoffice-3-4.]
More information about the LibreOffice