eff80a9fd990de36 (Allow custom "comment char", 2013-01-16) added general core.commentchar support but forgot to update git-rebase--interactive to respect it. 180bad3d10fe3a7f (rebase -i: respect core.commentchar, 2013-02-11) addressed this oversight but missed one instance of hard-coded '#' comment character in skip_unnecessary_picks(). Fix this. 9a46c25bdbf79744 (rebase: interactive: fix short SHA-1 collision, 2013-08-12) added another instance of hard-coded '#' comment character in transform_todo_ids(). Fix this, as well. Signed-off-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx> --- While planning to work on a couple patches to address Junio's comments on [1] (currently in 'next'), I realized that that series introduced a small regression by ignoring core.commentchar and assuming that the todo list comment character is unconditionally '#'. This patch fixes that regression. While fixing the regression, I also noticed that the original patch (180bad3d10fe3a7f) which added core.commentchar support to rebase -i missed one instance. This patch fixes that, as well. [1]: http://thread.gmane.org/gmane.comp.version-control.git/232146 git-rebase--interactive.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh index ea11e62..f246810 100644 --- a/git-rebase--interactive.sh +++ b/git-rebase--interactive.sh @@ -671,7 +671,7 @@ skip_unnecessary_picks () { ;; esac ;; - 3,#*|3,) + 3,"$comment_char"*|3,) # copy comments ;; *) @@ -693,7 +693,7 @@ transform_todo_ids () { while read -r command rest do case "$command" in - '#'* | exec) + "$comment_char"* | exec) # Be careful for oddball commands like 'exec' # that do not have a SHA-1 at the beginning of $rest. ;; -- 1.8.4.rc3.500.gc3113b0 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html