From: Sven Verdoolaege <skimo@liacs.nl>
Date: Fri, 24 Jun 2005 16:17:18 GMT
Subject: Re: rev-parse, unknown arguments and extended sha1's
Message-ID: <20050624161718.GA14909@pc117b.liacs.nl>
In-Reply-To: <Pine.LNX.4.58.0506240904240.11175@ppc970.osdl.org>

On Fri, Jun 24, 2005 at 09:09:23AM -0700, Linus Torvalds wrote:
> Anyway, add the --revs-only, and the test against empty should make a bit 
> more sense. In fact, it _should_ test that it's a SHA1 (and not a rev 
> argument), but I couldn't come up with a good way to test that in shell.
> 

This is what I use:

rev2=`echo $rev | sed -e 's/[^0-9a-f]//'`
rev3=`echo $rev2 | sed -e 's/........................................//'`
if test "x$rev3" = "x" -a "x$rev" = "x$rev2"; then
	echo "valid SHA1"
fi

skimo