threads / patch / 11894

patchgit-rebase.sh: Read $git_am_opt user setting inside --continue option

Subject: [PATCH] git-rebase.sh: Read $git_am_opt user setting inside --continue option

## tl;dr

2 messages between Feb 5, 2008 and Feb 6, 2008. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Jari Aalto· Feb 5, 2008, 13:56 UTC · lore
Honor user settable --whitespace=<control>
Signed-off-by: Jari Aalto <jari.aalto AT cante.net>
---
 git-rebase.sh |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
Show changes to git-rebase.sh +2 −1
diff --git a/git-rebase.sh b/git-rebase.sh
index bdcea0e..ec052c4 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -174,7 +174,8 @@ do
 		head_name=$(cat .dotest/head-name) &&
 		onto=$(cat .dotest/onto) &&
 		orig_head=$(cat .dotest/orig-head) &&
-		git am --resolved --3way --resolvemsg="$RESOLVEMSG" &&
+		git am $git_am_opt --resolved --3way \
+		    --resolvemsg="$RESOLVEMSG" &&
 		move_to_original_branch
 		exit
 		;;
-- 
1.5.4-rc5.GIT-dirty


-- 
Welcome to FOSS revolution: we fix and modify until it shines
Junio C Hamano· Feb 6, 2008, 02:26 UTC · re: Jari Aalto · lore

Re: [PATCH] git-rebase.sh: Read $git_am_opt user setting inside --continue option

Jari Aalto <jari.aalto@cante.net> writes:
Show 21 quoted lines
> Honor user settable --whitespace=<control>
>
> Signed-off-by: Jari Aalto <jari.aalto AT cante.net>
> ---
>  git-rebase.sh |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/git-rebase.sh b/git-rebase.sh
> index bdcea0e..ec052c4 100755
> --- a/git-rebase.sh
> +++ b/git-rebase.sh
> @@ -174,7 +174,8 @@ do
>  		head_name=$(cat .dotest/head-name) &&
>  		onto=$(cat .dotest/onto) &&
>  		orig_head=$(cat .dotest/orig-head) &&
> -		git am --resolved --3way --resolvemsg="$RESOLVEMSG" &&
> +		git am $git_am_opt --resolved --3way \
> +		    --resolvemsg="$RESOLVEMSG" &&
>  		move_to_original_branch
>  		exit
>  		;;

Actually this is a bug in git-am. am is supposed to remember the original command line options.

8c31cb8 (git-am: --whitespace=x option) does it correctly, but later 67dad68 (add -C[NUM] to git-am) broke it and other changes inherited the breakage.

← back to recent threads