Wireshark mailing list archives
Re: Gerrit patches with trailing whitespace
From: Bill Meier <wmeier () newsguy com>
Date: Mon, 25 Aug 2014 13:59:33 -0400
On 8/25/2014 1:48 PM, Alexis La Goutte wrote:
Should we add some info then to the Dev Guide as to where to get the hook,It's already in http://wiki.wireshark.org/Development/SubmittingPatches#Setup We keep running into this problem - should the wiki page and the dev guide be consolidated?and also run a server-side hook to reject the push?+1 if we can figure out how to return a nice error message explanation and not just "Your change was rejected by the remote server."-1 or only trailing whitespace check for the moment... ;-)
I agree with Alexis.The hook needs work, which I started to do, but then I guess I lost interest when the work got to be "bigger than a breadbox" and then didn't announce that I was no longer working on making changes to the
hook. Bill ___________________________________________________________________________ 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 patches with trailing whitespace Graham Bloice (Aug 25)
- Re: Gerrit patches with trailing whitespace Evan Huus (Aug 25)
- Re: Gerrit patches with trailing whitespace Juan Jose Martin Carrascosa (Aug 25)
- Re: Gerrit patches with trailing whitespace Graham Bloice (Aug 25)
- Re: Gerrit patches with trailing whitespace Graham Bloice (Aug 25)
- Re: Gerrit patches with trailing whitespace Evan Huus (Aug 25)
- Re: Gerrit patches with trailing whitespace Alexis La Goutte (Aug 25)
- Re: Gerrit patches with trailing whitespace Bill Meier (Aug 25)
- Re: Gerrit patches with trailing whitespace Juan Jose Martin Carrascosa (Aug 25)
- Re: Gerrit patches with trailing whitespace Evan Huus (Aug 25)
- Re: Gerrit patches with trailing whitespace Graham Bloice (Aug 25)
- <Possible follow-ups>
- Re: Gerrit patches with trailing whitespace Peter Kjellerstedt (Aug 26)