From: Andy Dougherty Date: Tue, 18 Dec 2007 13:35:53 GMT Subject: Re: [PATCH] Minor portability patch to git-submodule Message-ID: In-Reply-To: On Mon, 17 Dec 2007, Johannes Schindelin wrote: > On Mon, 17 Dec 2007, Andy Dougherty wrote: > > > - git ls-files --stage -- "$@" | grep -e '^160000 ' | > > + git ls-files --stage -- "$@" | egrep -e '^160000 ' | > > Nack. egrep is not available on all platforms. But then I have to wonder > why not saying "grep '^160000 '" instead? Your last suggestion is easily and obviously better -- I'll assume you don't need an explicit patch and can just hand-edit mine. Still, I'd have thought egrep was fine. As far as I recall, it goes back to v7 Unix. Or are there non-unix systems at issue (perhaps cygwin variants or something) that have grep but not egrep? Anyway, thanks. -- Andy Dougherty doughera@lafayette.edu