Re: [PATCH] parse-remote: replace unnecessary sed invocation
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:50:56
Stephen Boyd [off-list ref] writes:
On Wed, Mar 30, 2011 at 11:31 AM, Michael Schubert [off-list ref] wrote:quoted
quoted
+ curr_branch=$(git symbolic-ref -q HEAD) + curr_branch="${cur_branch#refs/heads/}"^^^Ah! How did this pass the test suite... Anyway, looks like Junio applied this patch to maint already :-( Sorry.
Heh, I was wondering what those three carets under ads in "#refs/heads/" was pointing at, and didn't see you had only one R in the variable name. Thanks for a correction, and Michael, please be a bit more specific (and point the correct part with your carret) when you send your messages.
quoted hunk
Junio, please apply this (probably word wrapped since this is gmail interface) and drop your trust level of me by a few points.diff --git a/git-parse-remote.sh b/git-parse-remote.sh index 9168879..ea093d2 100644 --- a/git-parse-remote.sh +++ b/git-parse-remote.sh@@ -6,7 +6,7 @@ GIT_DIR=$(git rev-parse -q --git-dir) || :; get_default_remote () { curr_branch=$(git symbolic-ref -q HEAD) - curr_branch="${cur_branch#refs/heads/}" + curr_branch="${curr_branch#refs/heads/}" origin=$(git config --get "branch.$curr_branch.remote") echo ${origin:-origin} }