Wireshark mailing list archives
Re: Gerrit Merge "<gerrit topic>" commits
From: Evan Huus <eapache () gmail com>
Date: Thu, 30 Jan 2014 16:45:57 -0500
On Thu, Jan 30, 2014 at 2:35 PM, Gerald Combs <gerald () wireshark org> wrote:
On 1/30/14 6:17 AM, Bálint Réczey wrote:Hi, 2014-01-30 Evan Huus <eapache () gmail com>:I believe the simpler answer is that the submit type has been set to "Merge If Necessary" which means if changes are not submitted exactly on top of the change they were authored on, Gerrit will produce a merge automatically.I suggested using "Cherry Pick" for "Change Submit Action" and I would like to bring it up again. BTW this is what is documented on our workflow page. http://wiki.wireshark.org/Development/Workflow : "After all the criteria are met any Core Developer can push the "Submit Change" button; integration to the target branch does not happen automatically. Changes are cherry-picked to the target branch by Gerrit to keep the history linear. The rationale behind this is that while merging would reflect the history more accurately, linear history is easier to interpret by humans. Changes with dependencies are cherry-picked after all dependencies are cherry-picked, too. When a cherry-pick can't be done without conflict, the change needs to be updated in the review. " This is how I set up Gerrit at my previous job, too.I changed the "Change Submit Action" for Wireshark to "Cherry Pick". In case anyone is interested the official documentation is at http://test.code.wireshark.org/review/Documentation/project-setup.html#submit_type
How does test.code.wireshark... compare to just code.wireshark...? Is the testbed still up for a reason, or do they both resolve to the same place now? https://code.wireshark.org/review/Documentation/project-setup.html#submit_type
and a discussion of the various options on the repo-discuss (Gerrit) Google group is at https://groups.google.com/forum/#!topic/repo-discuss/GInKJgLUa2w
Since that discussion is too old to include "Rebase if Necessary", I'm curious what the pros/cons of that are (do you know, Balint?). It seems like it's got the pros of "Cherry Pick" in that it maintains linear history etc, without all the complications of ignoring dependencies and rewriting commit messages. ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Gerrit Merge "<gerrit topic>" commits Roland Knall (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Evan Huus (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Bálint Réczey (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Alexis La Goutte (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Gerald Combs (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Evan Huus (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Bálint Réczey (Jan 31)
- Re: Gerrit Merge "<gerrit topic>" commits Bálint Réczey (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Evan Huus (Jan 30)
- Re: Gerrit Merge "<gerrit topic>" commits Roland Knall (Jan 30)