osmith has posted comments on this change. ( https://gerrit.osmocom.org/11560 )
Change subject: replace src/* git scripts with a single src/gits
......................................................................
Patch Set 11: Verified+1 Code-Review+1
> my --porcelain --long seems to be identical to just status:
I was wondering if --porcelain --long changed the output to "up-to-date" (with
dashes) in your git version as well. But it seemed not to be the case because
you did not modify the string to compare against.
So I've added the version with dashes too, otherwise "gits status" is broken
for me (patchset 9) and removed "--procelain" because it's irritating when it
has no effect (patchset 11).
> That said, if you want to change it to argparse so that it works and looks
> good, do go ahead in a follow-up patch. ok?
Done in patchset 10. I think it also looks good with argparse now, what do you
think?
$ ./gits -h
usage: gits [-h] {status,st,s,fetch,f,rebase,r,re,sh,do} ...
gits: conveniently manage several git subdirectories. Instead of doing the 'cd
foo; git status; cd ../bar; git status' dance, this helps to save your time
with: status, fetch, rebase, ...
optional arguments:
-h, --help show this help message and exit
action:
{status,st,s,fetch,f,rebase,r,re,sh,do}
status (st, s) show a branch summary and indicate modifications
fetch (f) run 'git fetch' in each clone (use before rebase)
rebase (r, re) interactively ff-merge master, rebase current branches
sh run shell command in each clone (`gits sh echo hi`)
do run git command in each clone (`gits do clean -dxf`)
--
To view, visit https://gerrit.osmocom.org/11560
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-dev
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I579e7af26d76d5c5d83b2349695456bc7b54f5a2
Gerrit-Change-Number: 11560
Gerrit-PatchSet: 11
Gerrit-Owner: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Comment-Date: Thu, 08 Nov 2018 10:46:59 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
