[PATCH 02/16] email-lda: always use the cover letter

Keller, Jacob E jacob.e.keller at intel.com
Thu Feb 6 13:18:11 PST 2014


On Thu, 2014-02-06 at 18:30 +0200, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <artem.bityutskiy at linux.intel.com>
> 
> The previous patch from Jacob only works when there is more than one patch in
> the series, but did not work for the situation when there is a cover letter
> (0/1) and a single patch (1/1). This patch fixes the situation.
> 
> Signed-off-by: Artem Bityutskiy <artem.bityutskiy at linux.intel.com>
> ---
>  email/aiaiai-email-lda | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/email/aiaiai-email-lda b/email/aiaiai-email-lda
> index 3b102be..446b32a 100755
> --- a/email/aiaiai-email-lda
> +++ b/email/aiaiai-email-lda
> @@ -344,13 +344,11 @@ process_mbox()
>  		{ reject "$mbox" "\"$prefix_format\" prefix not found"; return; }
>  
>  	# If the patch prefix contains m/n, fetch m and n.
> -	local m
> -	local n
> +	local m n
>  	m="$(subject_m "$subj")"
>  	n="$(subject_n "$subj")"
>  
> -	if [ -z "$m" ] || [ "$n" -eq 1 ]; then
> -		[ "$m" != "0" ] || { message "Dropping patch \"0/1\""; return; }
> +	if [ -z "$m" ]; then
>  		message "Queueing stand-alone patch \"$subj\""
>  		queue_mboxfile
>  	else

Oh. Nice catch!

Thanks,
Jake


More information about the aiaiai mailing list