Date: Thu, 30 Aug 2012 23:49:50 -0400
From: Austin Clements <amdragon@MIT.EDU>
To: david@tethera.net
Subject: Re: [PATCH] test: canonicalize content-type in "Sending a message
	via (fake) SMTP"
Message-ID: <20120831034934.GG11179@mit.edu>
References: <20120830013653.GF11179@mit.edu>
	<1346369074-8366-1-git-send-email-david@tethera.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1346369074-8366-1-git-send-email-david@tethera.net>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: notmuch@notmuchmail.org, David Bremner <bremner@debian.org>
Precedence: list

LGTM.

Quoth david@tethera.net on Aug 30 at  8:24 pm:
> From: David Bremner <bremner@debian.org>
> 
> The version of message.el in emacs24 omits the charset=us-ascii,
> causing the current version of this test to fail. With this patch, we
> accept either option.  According to RFC 2046, they are semantically
> equivalent.
> ---
> 
> as discussed on IRC, the $ is not quite as nice as \b, but it is
> POSIX.2, at least according to "man 7 regex"
> 
>  test/emacs |    5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/test/emacs b/test/emacs
> index afe35ba..5d118b6 100755
> --- a/test/emacs
> +++ b/test/emacs
> @@ -169,7 +169,8 @@ emacs_deliver_message \
>       (insert "To: user@example.com\n")'
>  sed \
>      -e s',^User-Agent: Notmuch/.* Emacs/.*,User-Agent: Notmuch/XXX Emacs/XXX,' \
> -    -e s',^Message-ID: <.*>$,Message-ID: <XXX>,' < sent_message >OUTPUT
> +    -e s',^Message-ID: <.*>$,Message-ID: <XXX>,' \
> +    -e s',^\(Content-Type: text/plain\); charset=us-ascii$,\1,' < sent_message >OUTPUT
>  cat <<EOF >EXPECTED
>  From: Notmuch Test Suite <test_suite@notmuchmail.org>
>  To: user@example.com
> @@ -178,7 +179,7 @@ Date: 01 Jan 2000 12:00:00 -0000
>  User-Agent: Notmuch/XXX Emacs/XXX
>  Message-ID: <XXX>
>  MIME-Version: 1.0
> -Content-Type: text/plain; charset=us-ascii
> +Content-Type: text/plain
>  
>  This is a test that messages are sent via SMTP
>  EOF