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

On Tue, Apr 09, 2013 at 02:00:52PM -0700, Junio C Hamano wrote:

John Keeping [off-list ref] writes:

quoted

+	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.

quoted

 	(
 		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).