Re: [PATCH 0/4] contrib/subtree: general updates

2013-03-23 Thread Paul Campbell
On Thu, Mar 14, 2013 at 3:30 AM, Junio C Hamano wrote: > Miles Bader writes: > >> Paul Campbell writes: >>> James and Michael's patches add if clauses that use the >>> bashism 'if []' rather than 'if test'. >> >> "Bashism"...? I dunno how portable is, but "[" is an old unix alias >> for "test"

Re: [PATCH 0/4] contrib/subtree: general updates

2013-03-13 Thread Junio C Hamano
Miles Bader writes: > Paul Campbell writes: >> James and Michael's patches add if clauses that use the >> bashism 'if []' rather than 'if test'. > > "Bashism"...? I dunno how portable is, but "[" is an old unix alias > for "test" ... it certainly predates bash... Correct. [[ ... ]] is new and

Re: [PATCH 0/4] contrib/subtree: general updates

2013-03-13 Thread Miles Bader
Paul Campbell writes: > James and Michael's patches add if clauses that use the > bashism 'if []' rather than 'if test'. "Bashism"...? I dunno how portable is, but "[" is an old unix alias for "test" ... it certainly predates bash... -miles -- Occam's razor split hairs so well, I bought the w