Re: [PATCH v2 3/6] update_unicode.sh: pin the uniset repo to a known good commit

2016-12-15 Thread Junio C Hamano
Dennis Kaarsemaker writes: > On Wed, 2016-12-14 at 00:31 +0100, Beat Bolli wrote: >> +   ( cd uniset && git checkout 4b186196dd ) > > Micronit, but this is perhaps better written as > > git -C uniset checkout 4b186196dd > > to avoid the subshell and cd. > > D. In the context of this script,

Re: [PATCH v2 3/6] update_unicode.sh: pin the uniset repo to a known good commit

2016-12-15 Thread Dennis Kaarsemaker
On Wed, 2016-12-14 at 00:31 +0100, Beat Bolli wrote: > +   ( cd uniset && git checkout 4b186196dd ) Micronit, but this is perhaps better written as git -C uniset checkout 4b186196dd to avoid the subshell and cd. D.