From: Chris Packham Date: Fri, 05 Sep 2014 10:06:48 GMT Subject: [RFC PATCHv3 1/4] am: avoid re-directing stdin twice Message-ID: <1409911611-20370-2-git-send-email-judge.packham@gmail.com> In-Reply-To: <1409911611-20370-1-git-send-email-judge.packham@gmail.com> In check_patch_format we feed $1 to a block that attempts to determine the patch format. Since we've already redirected $1 to stdin there is no need to redirect it again when we invoke tr. This prevents the following errors when invoking git am $ git am patch.patch tr: write error: Broken pipe tr: write error Patch format detection failed. Cc: Stephen Boyd Signed-off-by: Chris Packham --- Nothing new since http://article.gmane.org/gmane.comp.version-control.git/256425 git-am.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/git-am.sh b/git-am.sh index ee61a77..fade7f8 100755 --- a/git-am.sh +++ b/git-am.sh @@ -250,7 +250,7 @@ check_patch_format () { # discarding the indented remainder of folded lines, # and see if it looks like that they all begin with the # header field names... - tr -d '\015' <"$1" | + tr -d '\015' | sed -n -e '/^$/q' -e '/^[ ]/d' -e p | sane_egrep -v '^[!-9;-~]+:' >/dev/null || patch_format=mbox -- 2.1.0.64.gc343089