What does "Do not merge" mean in some of the Android git commits?

2,667 views
Skip to first unread message

Mac Wang

unread,
Dec 16, 2011, 8:18:02 PM12/16/11
to android-...@googlegroups.com
Hi,

I'm curious about it for long time. I saw a lot of Android git commits were labeled as "Do not merge". But they are merged in to some other branch. (ex. ics-mr1).

So what does it really mean in Google internally?

Mac

Jean-Baptiste Queru

unread,
Dec 16, 2011, 8:54:48 PM12/16/11
to android-...@googlegroups.com
We (Google) routinely develop on multiple branches at the same time.
In order to make sure that the later branch (e.g. ics-mr1) contains
all the new features and bugfixes developed in an older branch (e.g.
ics-mr0), we have a server that automatically takes every commit made
in ics-mr0 and merges it into ics-mr1.

However, sometimes the engineer making a change in ics-mr0 knows that
this change doesn't apply to ics-mr1, e.g. because a similar issue was
fixed differently in ics-mr1 and the fix from ics-mr0 wasn't
necessary. In that case, the engineer includes the words "do not
merge" in their change description, and the auto-merger performs a
"git merge -s ours" instead of "git merge" when handling that change.

There's a bit more complexity involved, but that's the high-level view.

JBQ

> --
> You received this message because you are subscribed to the "Android
> Building" mailing list.
> To post to this group, send email to android-...@googlegroups.com
> To unsubscribe from this group, send email to
> android-buildi...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/android-building?hl=en

--
Jean-Baptiste M. "JBQ" Queru
Software Engineer, Android Open-Source Project, Google.

Questions sent directly to me that have no reason for being private
will likely get ignored or forwarded to a public forum with no further
warning.

Reply all
Reply to author
Forward
0 new messages