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 gits...@pobox.com wrote: Miles Bader mi...@gnu.org writes: Paul Campbell pcampb...@kemitix.net 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 [

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

2013-03-13 Thread Paul Campbell
Less ambitious patch series this time. James and Michael's patches add if clauses that use the bashism 'if []' rather than 'if test'. I've left them as they are to match the other if clauses within the git-subtree.sh file. I remember reading a comment on the list about matching the local style

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

2013-03-13 Thread Miles Bader
Paul Campbell pcampb...@kemitix.net 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,