From: Jameson Graef Rollins <jrollins@finestructure.net>
To: Notmuch Mail <notmuch@notmuchmail.org>
Subject: problems with message/rfc822 parts
User-Agent: Notmuch/0.6 (http://notmuchmail.org) Emacs/23.3.1
	(x86_64-pc-linux-gnu)
Date: Thu, 02 Jun 2011 10:13:51 -0700
Message-ID: <87sjrsi2b4.fsf@servo.factory.finestructure.net>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha256; protocol="application/pgp-signature"
Precedence: list

--=-=-=

Hey, folks.  I've been noticing a couple of issues with message/rfc822
part handling in recent builds of notmuch/master.  They are rooted in
the new part handling rework that was done recently.  I just want to
mention them here, to make people aware of them, until we get a chance
to address them.

The first issue is that show --format=raw --part on a message/rfc822
part is throwing a gmime exception, even though notmuch is still
producing output and returning successfully.  I just sent a patch to the
list addressing the issues.  However, the test suite was apparently
unable to catch that something unexpected was being output to stderr, so
we still need to address that.

Another issue is that we're no longer outputting any of the headers of
message/rfc822 parts.  This shouldn't be too hard to hack back in, so
hopefully I can get to that this weekend.

There was also a problem with the emacs display of message/rfc822 parts,
and I just sent a patch to the list about that as well.

However, there is a larger issue related to the ones above which is that
message parts in general are not being handled consistently, including
the top level message.  amdragon and I were actually discussing this on
irc the other day.  A possible solution is to make a general
message-part-handling function, that handles output of headers and the
message body, and call that function both on the top level message and
on any message sub parts.  I think that would make things a lot cleaner,
and would make the output more consistent and intuitive.

A somewhat related issue is that the "raw" format for parts is not
really doing the expected thing in certain cases, particularly with
regards to message parts.  dkg and I sort of realized this when we were
reworking the part handling a couple weeks ago.  One expects, rightfully
so, that show --format=raw --part on a message part would output the
full, raw, well-formatted rfc822 message enclosed in that part.
Currently that is not the case, since, for instance, the headers are not
output and the part handling is recursing over all the message
sub-parts.

I don't know necessarily know the right solution for either of these
issues, although I have some ideas (assuming I don't run into gmime
limitations).  Feedback and suggestions would be wonderful.
Unfortunately I don't have time to work on any of these issues at the
moment (hopefully this weekend), but I just wanted to throw them out
there, in case anyone wants to tackle them before I get to them.

jamie.

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iQIcBAEBCAAGBQJN58TPAAoJEO00zqvie6q8ADgP/2TaIe0TzFYWnDPB0KsoAHgB
nSlD3CgFD8qolAY37Q4Oxoy1XqzaKZGJAYtFHErcEHvQb4vesxhXJYMjLpKVCyUK
zUzU6s0Qjy1t3RQ9pUZF1I4HsFkonMti1DdhMBiJ5B3hkGEYFLVQlgWqGq3Q7SYp
7HOG01bDNfR+nMS1ZqXCxWiuG3RLe10rVBO6gmT3s8UZdRsRBeRl8ZoNEX8zliED
/Fe14NG+CT2TVEdpKkhKpC5ER1zouMFQ9bnVBX+PypJfyYHuDgsr3zeH3U44eNGt
EN3917E+Vjl1IfRW0c62oaY2Lvd8Wu5Fi/F4OCIYi243Xh0Q7pe+vqvS/KhVJq6V
WhM2Dl1ahiNjy2tas3AUfVDyjwCgjFYqkvt+Fn/ekyrUGUHXrNsQylh3UeyxpgzX
fSrxvSr2tWYH5uSYJXv2AAtWBWwd7jeG3sZppgSJSBsA9B4k7f8ZDhYD7o1zREtN
k70ZE076rGmpi651liRrEJewK5zoKaRiZ8qj/2xqPtcZNT5GrvKtwa1u9reo1QP7
/e3y6mIy8jXqpTVe1+xfypX+rLipu6lHUV6cEJNC9yBjis0T3qIKhxzgVDSegRcf
MSJutIH5WolZU7AmigwfaqZ3VvnW0IJHXL9vVS//2O4pxUx912goGndxZSMBvUPG
Vd+g9ROQuSq/LvwB2z5G
=UbME
-----END PGP SIGNATURE-----
--=-=-=--