On 2012-12-18 at 14:06 -0800, Michael Elkins wrote:
> My current plan is to ensure that all the bugfixes that are 
> applied to the mutt packages in various OS distributions have been 
> rolled into the hg repo, then push out a 1.5.22 release.
> 
> I've found the patches from some distros here:
> 
> Fedora: http://pkgs.fedoraproject.org/cgit/mutt.git/tree/?h=f18
> Debian: 
> http://anonscm.debian.org/gitweb/?p=pkg-mutt/mutt.git;a=tree;f=debian/patches;h=001cba693d8880724697043c10d833be9e4db387;hb=master
> 
> I would appreciate some feedback either from the maintainers/users of 
> other distros to make sure I'm not missing any important bugs.

FreeBSD:
 * http://svnweb.freebsd.org/ports/head/mail/mutt/files/

> Note that right now I am focusing just on bugfixes, not feature 
> patches, so that a new release can be created soon.

Files matching glob extra-patch-* are feature patches and not covered
below.  Files matching glob patch-* are automatically applied always.

You should probably look at:
  patch-bdb
  patch-contrib-Makefile.am
  patch-imap-message.c
  patch-nbsp
  patch-pgp

In detail:
 * patch-02 : packaging; looks ignorable
   make/html changes; use lynx not elinks
 * patch-05 : feature / old-behaviour-restoration
   something with y/n responses and prompts
 * patch-08 : default Muttrc change
   "set followup_to=no"
 * patch-Makefile.am : packaging; looks ignorable
   mime.types do not install by default
 * patch-Makefile.am.doc : packaging; might be useful
   "Do not format manual during installation"
   Changes SUBDIRS definition ordering, moves doc/ to after ./
   Haven't investigated context, might make sure format is done during
   build, not install?
 * patch-bdb : ** useful, update BDB_VERSIONS
   Adds a bunch of versions to BDB_VERSIONS
 * patch-color-eol : feature
   Adds "color_after_eol" option
 * patch-configure.ac : packaging?
   Removes ANSI-compliance check 'AM_C_PROTOTYPES'
 * patch-contrib-Makefile.am : ** useful?
   Updates SAMPLES definition
 * patch-crypt-gpgme.c : ignore, backport of in-tree fix
   d626fb87c489 fix, minimalised actual logic
 * patch-date-conditional : feature
     dgc.deepif.1
     patch-1.5.0.ats.date_conditional.1
 * patch-doc-manual.xml.head : ignore, packaging
   switches a DTD to use local path instead of public URL
 * patch-examples : ignore, packaging
   Moves examples installation location
 * patch-gnutls-CN-validation : CVE-2011-1429 fix
   I strongly suspect this is already in-tree
 * patch-imap-message.c : ** might be needed
   http://bugs.mutt.org/3288
   The fix confirms that a pointer is not NULL, and is not in current
   source (but might have been fixed differently?)
 * patch-init.c : OS-specific, ignorable
   uses gethostname() instead of uname() on ancient FreeBSD
   FreeBSD pre 5.0 isn't supported, I'm surprised this patch is still
   there
 * patch-mktemp : probably ignorable
   Use mktemp() with templated filenames instead of the fixed pattern in
   mutt; technically correct, I'm guessing mutt does what it does for
   portability
 * patch-nbsp : ** feature / charset bug-correction, might want
   Handle non-breaking whitespace in UTF-8 mode
 * patch-pgp : ** fixes \r\n in PGP-handling
   The fix looks like something needed, but the approach taken is
   brute-force
 * patch-smime-self : feature, "smime_encrypt_self" option
 * patch-smime-sender : feature, "smime_dont_check_sender" option
 * patch-threadcomplete : feature
   Adds 'a' flag to pattern searches, wraps match_threadcomplete()

Regards,
-Phil

Reply via email to