Home page logo
/

wireshark logo Wireshark mailing list archives

Re: Proposed Gerrit workflow (was: Re: Notes from Sharkfest '13)
From: Bálint Réczey <balint () balintreczey hu>
Date: Sat, 22 Jun 2013 17:03:45 +0100

Hi Martin,

2013/6/22 Martin Kaiser <lists () kaiser cx>:
Thus wrote Bálint Réczey (balint () balintreczey hu):

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.

would that mean that even the most basic change needs peer review and
approval based on the process defined by gerrit?

I'm a bit worried that this doubles the time for such simple changes.  I
often see this in corporate environments where people don't correct
typos, misleading variable names, formatting etc. because they can't be
bothered with the administrative overhead.
I think it depends on the people involved. In an environment similar to what
you described I collected several small changes in short reviewable commits and
asked for peer review for the set together.

We can relax the rules for Core Developers to let them bypass the peer review,
but I did not want to include this exception in the first proposal.
Speaking of myself I would be OK with requiring peer review for all my commits,
but it is not a surprise since I wrote the first version of the proposal. ;-)

What I'm really looking forward to in the proposed Gerrit work-flow is
the ability of having my changes
tested on architectures I don't use _before_ applying them to the main branch.

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

  By Date           By Thread  

Current thread:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault