Hashar: I think your criticisms of git-review might be out of date. On my debian system, "sudo apt-get install git-review" was sufficient to install a version of git review which is adequate for all my needs (I have never noticed any of the "known issues" you mention). And "git review -s" is a one-step way to set up the proper remotes and etc and leave you ready to use it. I am in the habit of typing "git review -u" since I noticed that sometimes it doesn't update its remotes, but I'm guessing that's more habit than necessity. None of the other steps you describe seem to be necessary. --scott
On Thu, Nov 12, 2015 at 4:10 AM, Antoine Musso hashar+wmf@free.fr wrote:
Le 12/11/2015 03:25, S Page a écrit :
On Wed, Nov 11, 2015 at 12:50 AM, Petr Bena benapetr@gmail.com wrote:
Ok, I will try to merge all useful stuff in here: https://www.mediawiki.org/wiki/Git_for_dummies
The problem is these are matters of widely-varying taste and background. When I tried to clean up in early 2013, git experts didn't even agree on whether the gerrit remote should be origin, or whether people should use `git review` at all.
Hello,
git-review is a python utility that acts as a thin wrapper around Gerrit workflow. It has been written by the OpenStack community which is a python shop.
It comes with few issues:
- the versions that comes with Linux distributions are fairly outdated
and comes with known issues.
- installing using the python package managers is not straightforward
and has several permissions issues depending on your operating system (some install system wide which require root, others to a user writable dir etc)
- old versions were pushing to the HEAD of the repo iirc which is
troublesome when you are working on another branch
- some global configuration is needed
- you need a remote named gerrit
On that last point, git-review 1.26 comes with a new option 'usepushurl' which makes git-review reuse the origin repo and just set the push url to the ssh:// url. The 'gerrit' remote will no more be needed.
Ie in your ~/.gitconfig :
[gitreview] usepushurl = 1
What I did until that new version is that all my clones were done with a remote named 'gerrit' (git clone -o gerrit <url>).
Since folks are tired of debugging python stacktraces and incorrect git-review configuration, some are recommending to use the underlying Gerrit workflow command:
git push origin HEAD:refs/publish/<BRANCH>[/<TOPIC>]
-- Antoine "hashar" Musso
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l