mercurial/crew: 9 outgoing changesets

Alexis S. L. Carvalho alexis at cecm.usp.br
Tue Jun 27 08:15:27 CDT 2006


Thus spake Mercurial Commits:
> http://hg.intevation.org/mercurial/crew?cs=158d3d2ae070
> changeset:   2504:158d3d2ae070
> parent:      2502:18cf95ad3666
> user:        Vadim Gelfer <vadim.gelfer at gmail.com>
> date:        Tue Jun 27 00:09:13 2006 -0700
> summary:     import: parse email messages

This is a nice thing to have.  A few comments:

- the Subject of the message was ignored when I tried to import a
  message with an inline patch - I haven't tried it with attachments

- it'd be nice to clean up the subject a bit - specifically remove
  leading spaces, "Re:", "[listname]" and maybe "[PATCH 17/42]", being
  careful with stuff like "Re: [listname] Re: [PATCH]  patch description"

- the docs mention the "Sender and subject line of email" - I think it'd
  be better to mention something like the "The From and Subject headers
  of the email": "Sender" is also a valid header that is added by e.g.
  mailing lists.

Alexis


More information about the Mercurial mailing list