Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Inline CPAN distribution.

Report information
The Basics
Id: 73448
Status: resolved
Priority: 0/
Queue: Inline

People
Owner: Nobody in particular
Requestors: cpan698301 [...] mstier.de
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Inline::C runs Makefile.PL in non-clean environment
The tests fail when PERL_INSTALL_ROOT env var is set. My best guess is that Makefile.PL should not be run without cleaning the environment first.
On Thu Dec 22 18:39:44 2011, https://www.google.com/accounts/o8/id?id=AItOawkyOHx--RP0sCuxSqimRLgFJ_YPC6QRWZk wrote: Show quoted text
> The tests fail when PERL_INSTALL_ROOT env var is set. My best guess is > that Makefile.PL should not be run without cleaning the environment first.
If it's just the PERL_INSTALL_ROOT env var that's causing the problem, perhaps I can just delete() that key (and its value) from %ENV in each and every one of the test scripts. (I gather it must play some role during installation of the module - in which case it should hopefully be fine to remove it in the test scripts.) What does this variable actually do ? Who uses it ? And to what sort of value is it normally set ? Cheers, Rob
From: cpan698301 [...] mstier.de
ExtUtils::Install PERL_INSTALL_ROOT is basically the equivalent to "make DESTDIR=...". From what I saw, Inline::C uses ExtUtils/Module::Build or so to temporarily build Perl modules. You don't need to clean the environment for your modules -- just when invoking the perl module build process.
RT-Send-CC: cpan698301 [...] mstier.de
On Thu Jan 12 19:59:17 2012, https://www.google.com/accounts/o8/id?id=AItOawkyOHx--RP0sCuxSqimRLgFJ_YPC6QRWZk wrote: Show quoted text
> PERL_INSTALL_ROOT is basically the equivalent to "make DESTDIR=...".
Thanks. In the next release of Inline, the need to make provision for PERL_INSTALL_ROOT will be documented in Inline.pod and affected test scripts will start with something like: ################### BEGIN { if (exists $ENV{PERL_INSTALL_ROOT}) { warn "\nIgnoring \$ENV{PERL_INSTALL_ROOT} in $0\n"; delete $ENV{PERL_INSTALL_ROOT}; } }; ################### This will allow the tests to pass without - and still allow Inline itself to be installed in the user's intended location. Cheers, Rob
Fixed in 0.50. Thanks for the report. Cheers, Rob