Subject: Re: [PATCH] rev-parse --show-prefix: add in trailing newline

From: Andreas Schwab <[email-masked]>
Date: Tue, 10 Apr 2012 16:41:45 +0200

Junio C Hamano <[email-masked]> writes:

> Because the assignment will strip the trailing LF away

No, it doesn't.  Command substitution does.

> But it will break existing scripts if they expect the following to work:
>
> 	file="$(git rev-parse --show-prefix)/$1"
>         cd_to_toplevel
>         test -f "$file"

This is already broken because of the extra slash.

Andreas.

-- 
Andreas Schwab, [email-masked]
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."