Bug #72223 for MIME-tools: Missing dependency: Convert-Binhex

Sun Nov 06 16:42:15 2011 tk [...] giga.or.at - Ticket created

Subject:	Missing dependency: Convert-Binhex
Date:	Sun, 6 Nov 2011 22:42:05 +0100
To:	bug-MIME-tools [...] rt.cpan.org
From:	Thomas Klausner <tk [...] giga.or.at>

lib/MIME/Decoder/BinHex.pm has the line: use Convert::BinHex; but the Makefile doesn't note the dependency; it also isn't used in any tests, so as an end user you only find the problem when an application like amavisd-new calls the code. Please include Convert::BinHex in the dependency list in the Makefile. For bonus points, include a test for the code using it :) Thanks, Thomas

Mon Nov 21 13:29:56 2011 dfs+pause [...] roaringpenguin.com - Taken

Mon Nov 21 13:33:23 2011 dfs [...] roaringpenguin.com - Correspondence added

Subject:	Re: [rt.cpan.org #72223] Missing dependency: Convert-Binhex
Date:	Mon, 21 Nov 2011 13:33:05 -0500
To:	bug-MIME-tools [...] rt.cpan.org
From:	"David F. Skoll" <dfs [...] roaringpenguin.com>

On Sun, 6 Nov 2011 16:42:16 -0500 "Thomas Klausner via RT" <bug-MIME-tools@rt.cpan.org> wrote: Show quoted text

> Please include Convert::BinHex in the dependency list in the Makefile. > For bonus points, include a test for the code using it :)

Hmm. Convert::BinHex is really a soft dependency. I hesitate to make it a hard dependency because BinHex encoding is not recommended by any MIME standards-track RFC and many people never need to process BinHex-encoded files. I'll put in a "recommends" clause in the Makefile. And I guess I should eventually write a test. :) Regards, David.

Mon Nov 21 13:33:24 2011 The RT System itself - Status changed from 'new' to 'open'

Fri Jun 08 10:00:27 2012 dfs+pause [...] roaringpenguin.com - Correspondence added

Hi, I have just uploaded MIME-tools-5.503 to CPAN, which I believe resolves this ticket. Regards, David.

Fri Jun 08 10:00:28 2012 dfs+pause [...] roaringpenguin.com - Status changed from 'open' to 'resolved'