mutt-devel @1.5.21 +sidebar "Could not create temporary file!"

[Sylvain.Soliman+macports@…]

A missing closedir() in the old Debian sidebar patch we use results in consuming all file descriptors when handling Maildir.

See ​http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=610191 for the corresponding Debian bug-report. I get the exact same symptoms on my Snow Leopard MBA (mostly the temp file message, sometimes the missing file descriptors message).

I think we simply need to get the fixed version of their sidebar patch (i.e. 1.5.21_4 or later) [e.g. ​http://patch-tracker.debian.org/patch/series/view/mutt/1.5.21-5/mutt-patched/sidebar ]

[kchr@…]

I can confirm that this is still happening @1.5.21_2.

[kchr@…]

Cc Me!

[kchr@…]

According to the Debian bug report aforementioned, the bug was fixed in 1.5.21-4: {{

From: Antonio Radici Date: Tue, 12 Apr 2011 06:19:03 +0000

fixed 610191 1.5.21-4 thanks

I uploaded a version that fixes this yesterday night but unfortunately I put the wrong bug to close in the debian/changelog, this should be fine now.

Cheers Antonio

}}

[kchr@…]

I have compiled mutt-devel @1.5.21_2 from ports using the latest version of the sidebar patch from the Debian unstable repository. The patch can be found here:

​http://patch-tracker.debian.org/patch/series/dl/mutt/1.5.21-6.2/mutt-patched/sidebar

The fd leak seems to be fixed, at least at my end! Let me know if I can help out in any way.

[kchr@…]

Just contacted the maintainer of mutt-devel port.

[kchr@…]

I just learned "openmaintainer" means anyone can patch it. I am currently updating the Portfile to apply the latest sidebar patch from Debian, which have been working for two months now without any hitches.

[kchr@…]

Attaching an updated Portfile and a proper patch from the Debian team (revision 6.2).

Incremented port revision to 1, since the users that have installed this variant would not get notice about the update otherwise — and the bug could be sneaky to discover in some cases.

[kchr@…]

mutt-devel Portfile, revision 1

[kchr@…]

Updated sidebar patch, from mutt-patched in Debian repo (revision 6.2)

[mf2k (Frank Schima)]

Please instead attach a unified diff of the portfile against svn trunk so we can easily see what changes you made.

[kchr@…]

Portfile diff against svn trunk

[jon.armond@…]

Cc Me!

[neverpanic (Clemens Lang)]

Commited in r103397.