Bug #91407 for strictures: please add support for PUREPERL

Fri Dec 13 17:03:27 2013 djerius [...] cpan.org - Ticket created

Subject:

please add support for PUREPERL_ONLY

I've prepared a patch which adds support for PUREPERL_ONLY. You may find it here: https://github.com/djerius/strictures The code is 99% stolen from Devel::GlobalDestruction Rationale: I'm using cpanm --pp to populate a directory for App::FatPacker. strictures is required by the application. Prior to 1.005002 when compile-only modules were recommendations, cpanm happily ignored them, and I was able to install strictures in pure-perl mode. After 1.005002, when those modules became requirements if the host platform had a C compiler (which my dev box does), cpanm attempted to install them using PUREPERL_ONLY=1, and Lexical::SealRequireHints properly notified me that it must be compiled and the installation of strictures failed. This patch allows strictures to be installed under cpanm --pp, whether or not a C compiler is present. Thanks, Diab

Fri Dec 13 19:19:28 2013 haarg [...] haarg.org - Correspondence added

It does seem odd though that Lexical::SealRequireHints won't build in pure perl mode. Unless you are on 5.10.0, it should be fine to work without it. Probably worth filing an issue about. That doesn't really effect this issue though, since the extra checks need XS even if Lexical::SealRequireHints doesn't. I've merged this, and will release it soon after adding some extra checks.

Fri Dec 13 19:19:29 2013 The RT System itself - Status changed from 'new' to 'open'

Fri Dec 13 19:19:30 2013 haarg [...] haarg.org - Status changed from 'open' to 'patched'

Sat Aug 16 04:53:27 2014 haarg [...] haarg.org - Correspondence added

This was fixed in 1.005003

Sat Aug 16 04:53:27 2014 haarg [...] haarg.org - Status changed from 'patched' to 'resolved'

Sat Aug 16 04:53:28 2014 haarg [...] haarg.org - Fixed in 1.005003 added

Bug #91407 for strictures: please add support for PUREPERL_ONLY