Re: [PATCH v2 7/9] builtin/send-pack.c: Use option parsing API

2015-08-21 Thread Jeff King
On Wed, Aug 19, 2015 at 03:46:25PM -0400, Dave Borowitz wrote: > >> + unsigned dry_run = 0; > >> + unsigned send_mirror = 0; > >> + unsigned force_update = 0; > >> + unsigned quiet = 0; > >> + unsigned push_cert = 0; > >> + unsigned use_thin_pack = 0; > >> +

Re: [PATCH v2 7/9] builtin/send-pack.c: Use option parsing API

2015-08-19 Thread Dave Borowitz
On Wed, Aug 19, 2015 at 2:00 PM, Stefan Beller wrote: > On Wed, Aug 19, 2015 at 8:26 AM, Dave Borowitz wrote: >> The old option parsing code in this plumbing command predates this >> API, so option parsing was done more manually. Using the new API >> brings send-pack more in line with push, and a

Re: [PATCH v2 7/9] builtin/send-pack.c: Use option parsing API

2015-08-19 Thread Stefan Beller
On Wed, Aug 19, 2015 at 8:26 AM, Dave Borowitz wrote: > The old option parsing code in this plumbing command predates this > API, so option parsing was done more manually. Using the new API > brings send-pack more in line with push, and accepts new variants > like --no-* for negating options. > >

[PATCH v2 7/9] builtin/send-pack.c: Use option parsing API

2015-08-19 Thread Dave Borowitz
The old option parsing code in this plumbing command predates this API, so option parsing was done more manually. Using the new API brings send-pack more in line with push, and accepts new variants like --no-* for negating options. Signed-off-by: Dave Borowitz --- builtin/send-pack.c | 163 +