Re: [PATCH v5] git-am: allow e-mail file(s) as input
- From
Stephen Boyd <bebarino@gmail.com>
- Date
- Jul 16, 2009, 07:09 UTC
- Message-ID
- <4A5ED22B.6050101@gmail.com>
- In-Reply-To
- <eb6317e39369adc8d2594c35ee351b49aaadcc24.1247721562.git.nicolas.s.dev@gmx.fr>
Nicolas Sebrecht wrote:
Show 22 quoted lines
> diff --git a/git-am.sh b/git-am.sh
> index d64d997..2b55ddc 100755
> --- a/git-am.sh
> +++ b/git-am.sh
> @@ -162,6 +162,17 @@ check_patch_format () {
> return 0
> fi
>
> + # Then, accept what really looks like (series of) email(s).
> + # the first sed select headers but the folded ones
> + sed -e '/^$/q' -e '/^[[:blank:]]/d' "$1" |
> + # this one is necessary for the next 'grep -v'
> + sed -e '/^$/d' |
> + grep -v -E -e '^[A-Za-z]+(-[A-Za-z]+)*:' ||
> + {
> + patch_format=mbox
> + return 0
> + }
> +
> # otherwise, check the first few lines of the first patch to try
> # to detect its format
> {This fails t4150-am.sh #10 (am -3 -q is quiet). You should redirect the output of the sed and grep to /dev/null like Junio did in his "how about this" patch.
Also, writing some tests would be helpful.