Wireshark mailing list archives
Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13)
From: Bálint Réczey <balint () balintreczey hu>
Date: Fri, 28 Jun 2013 22:44:57 +0200
Hi Marc, 2013/6/28 Marc Petit-Huguenin <marc () petit-huguenin org>:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 06/28/2013 07:21 AM, Jeff Morriss wrote:On 06/22/13 06:47, Bálint Réczey wrote:Hi All, 2013/6/21 Marc Petit-Huguenin <marc () petit-huguenin org>:-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 06/20/2013 04:52 PM, Guy Harris wrote:On Jun 20, 2013, at 2:58 PM, Marc Petit-Huguenin <marc () petit-huguenin org> wrote:On 06/20/2013 02:17 PM, Gerald Combs wrote:Advantates: - I'm not sure that an in-house equivalent (e.g. Gerrit plus a private repository) would be better than what Github offers.Yes, Gerrit is better than github:Presumably you mean "Gerrit plus a private repository is better than github", as Gerrit, as far as I can tell, is just software that works with a Git repository.Yes, although managing repositories being what Gerrit do, Gerrit without a least one repository would be a very boring application.:-) I have started describing a Gerrit based workflow which IMO would fit to the project at http://wiki.wireshark.org/Development/Workflow . Please check it and share your opinion.It would probably be good to include how/where patch submitters would attach other attachments (e.g., sample captures) that go along with their patch (I spent a little time going through the Gerrit docs and didn't find anything like that). (A large portion of the patches we accept need to have a corresponding test capture file before anyone will accept it.)Gerrit does not replace the bug tracker, in fact it complements it, so the attachments would still be stored in the bug - unless they are used for something useful for the build, like regression testing, in which case they would be part of the patchset. It is possible to configure Gerrit to link automatically to the Bugzilla bugs: https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#_a_id_trackingid_a_section_trackingid
Thank you for the links. I have added the following to the proposal: "If you would like to provide an example capture file along with patch, please open a bug in BugZilla and refer to it in the commit message (... bug 1234 ...). By attaching the capture file to a bug will automatically become part of the regression suite and your new code will be tested very often. " Cheers, Balint ___________________________________________________________________________ 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:
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13), (continued)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Michael Tuexen (Jun 24)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Bálint Réczey (Jun 24)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Anders Broman (Jun 24)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Michael Tuexen (Jun 24)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Bálint Réczey (Jun 25)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Bálint Réczey (Jun 27)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Marc Petit-Huguenin (Jun 28)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Marc Petit-Huguenin (Jun 28)
- Re: Proposed Gerrit workflow Jeff Morriss (Jun 28)
- Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13) Bálint Réczey (Jun 28)