Wireshark-dev: Re: [Wireshark-dev] Gerrit sandbox available for testing

From: Marc Petit-Huguenin <marc@xxxxxxxxxxxxxxxxxx>
Date: Wed, 18 Sep 2013 16:01:55 -0700
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On 09/18/2013 03:54 PM, Gerald Combs wrote:
> A few months ago we discussed migrating from SVN to Git + Gerrit. I managed
> to get Gerrit[1] sandbox up and running at
> 
> http://test.code.wireshark.org/review/
> 
> Feel free to try it out and create as many test accounts and pull requests
> as you like. You can find a quick introduction at
> 
> http://test.code.wireshark.org/review/Documentation/intro-quick.html
> 
> So far it looks nice. Anyone can be upload changes for review and if you 
> have sufficient permissions you can bypass review and push changes directly
> into the repository:
> 
> http://test.code.wireshark.org/review/Documentation/user-upload.html
> 
> The only big caveat I've run into so far is that proposed commit messages
> must have a Change-Id field. A commit-msg hook is provided for generating
> Change-Ids from the command line but it doesn't appear to work very well in
> the GUIs I've tried (Qt Creator and Git Extensions).
> 
> Proposals also have to be pushed to a magic "refs/for/<branch/>" branch 
> name. I inadvertently bypassed review a couple of times by doing a default
> push, but that might just be me.

A good way to fix that is to run the following command, so a "git push" will
always push in the right destination:

git config --add remote.origin.push HEAD:refs/for/master


- -- 
Marc Petit-Huguenin
Email: marc@xxxxxxxxxxxxxxxxxx
Blog: http://blog.marc.petit-huguenin.org
Profile: http://www.linkedin.com/in/petithug
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)

iQIcBAEBCAAGBQJSOjDfAAoJECnERZXWan7EZh0QAN7MsrHPdt/RCS7PQSCy7Mj/
+ji3E8YIy1gcAFq9+r3a6Aro4Fg8oDPDLh9lTq+lfgvzG2oeJCg77chunydB8DLF
GS+JDkED+KBAcZSY1Y9KK1hkFQMYlE42/6bvNqiZZ5HEH6onb6sHSJSquUq0rb3v
oYfMnXKoQg7cjSRnPhJn8Fz4bcWJ8KHzAEyzdaIALhB9R90SjtDeUEou798sKtkC
QJEimdyzfdtKJOhKvJaamJTM5qTZuybt/dCebS7XqA/CVjmb4Hygpx0QjYk02i9m
4sPVaqbOOuHzJvLLdqO7vCk6yuA1ILKYSGqBsXLBtlDW6pibb3RdtbIwy4A3hmIs
OQVtm1cjMDrdbY40G5ZzYRutCF793vLEGNTnBYI6h/yAeLIh1FmjEfAOQ2WLRHOs
THWA1sxMioCUjP8kcdl67lqICfoisx0VFlSes6nxz2PVml8p/7kt/30VVX5g81OL
ouS4s6tWq5qTEPmGfrcMsM3gyvg09BuuMjzHCx/YnfYIrAKiM+h5pPHVFjIfnd8k
eDEN4Ej6jNTMAbFt/+l3DPyKoxM5HXyLWffem4EY6YasH8NI10L1INMbpALWNqG/
HxeB94i+Au/KPdb+2ZWtatSRvqU/J+CMAjPqRbIFIR5qcaALGOojX13KTypkOaWM
dl9/1rtCwDhMDPbaDntO
=TwYk
-----END PGP SIGNATURE-----