Subject: Re: [PATCH v2 2/2] submodule: drop the top-level requirement

From: John Keeping <[email-masked]>
Date: Tue, 9 Apr 2013 22:29:46 +0100

On Tue, Apr 09, 2013 at 02:00:52PM -0700, Junio C Hamano wrote:
> John Keeping <[email-masked]> writes:
> 
> > +	eval "set $(git rev-parse --sq --prefix "$wt_prefix" -- "$@")"
> 
> This may be handier than having to do the "for arg" loop git-am uses
> yourself.
> 
> >  	(
> >  		git ls-files --error-unmatch --stage -- "$@" ||
> >  		echo "unmatched pathspec exists"
> > @@ -335,6 +339,8 @@ cmd_add()
> >  		usage
> >  	fi
> >  
> > +	sm_path="$wt_prefix$sm_path"
> 
> But this is doing fine without "rev-parse --prefix" at all.

In this case we only have a single argument (and it must have a value).
In the cases using "rev-parse --prefix" we can have any number of
arguments (including zero).