[PATCH] rebase-am: fix regression with new cover-letter config
If you have a cover-letter configuration set to anything other than 'false', 'git format-patch' might generate a cover letter, and 'git am' doesn't seem to like that. Ideally 'git am' should skip the cover-letter, but for now lets just fix the regression. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- git-rebase--am.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/git-rebase--am.sh b/git-rebase--am.sh index 97f31dc..f84854f 100644 --- a/git-rebase--am.sh +++ b/git-rebase--am.sh @@ -31,8 +31,8 @@ else rm -f $GIT_DIR/rebased-patches git format-patch -k --stdout --full-index --ignore-if-in-upstream \ - --src-prefix=a/ --dst-prefix=b/ \ - --no-renames $root_flag $revisions $GIT_DIR/rebased-patches + --src-prefix=a/ --dst-prefix=b/ --no-renames --no-cover-letter \ + $root_flag $revisions $GIT_DIR/rebased-patches ret=$? if test 0 != $ret -- 1.8.2.1.643.ge3cc75d -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] rebase-am: fix regression with new cover-letter config
Felipe Contreras felipe.contre...@gmail.com writes: If you have a cover-letter configuration set to anything other than 'false', 'git format-patch' might generate a cover letter, and 'git am' doesn't seem to like that. Ideally 'git am' should skip the cover-letter, but for now lets just fix the regression. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- This is a follow-up to the fc/send-email-annotate topic, right? You are mean to say fix regression, which gave me a small heart attack ;-). I was worried if we had already merged it to 'next'. As I already wrote in the recent What's cooking, the series looks good to me. I am inclined to say that we should merge it (together with this update) to 'next'. Thanks for catching this before breaking the real world users. git-rebase--am.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/git-rebase--am.sh b/git-rebase--am.sh index 97f31dc..f84854f 100644 --- a/git-rebase--am.sh +++ b/git-rebase--am.sh @@ -31,8 +31,8 @@ else rm -f $GIT_DIR/rebased-patches git format-patch -k --stdout --full-index --ignore-if-in-upstream \ - --src-prefix=a/ --dst-prefix=b/ \ - --no-renames $root_flag $revisions $GIT_DIR/rebased-patches + --src-prefix=a/ --dst-prefix=b/ --no-renames --no-cover-letter \ + $root_flag $revisions $GIT_DIR/rebased-patches ret=$? if test 0 != $ret -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] rebase-am: fix regression with new cover-letter config
Felipe Contreras felipe.contre...@gmail.com writes: You are mean to say fix regression, which gave me a small heart attack ;-). I was worried if we had already merged it to 'next'. As I already wrote in the recent What's cooking, the series looks good to me. I am inclined to say that we should merge it (together with this update) to 'next'. Yeah, after thinking about it, it's not even really a regression, because people don't have format.coverletter=auto in their configs (yet), so the bug can't can possibly affect them. Here is what I tentatively queued. rebase-am: explicitly disable cover-letter If the user has a cover-letter configuration set to anything other than 'false', 'git format-patch' may generate a cover letter, which has no place in format-patch | am pipeline. The internal invocatoin of format-patch must explicitly override the configuration from the command line, just like --src-prefix and other options already do. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com Signed-off-by: Junio C Hamano gits...@pobox.com -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html