Skip Menu |

This queue is for tickets about the Module-Install CPAN distribution.

Report information
The Basics
Id: 53984
Status: resolved
Priority: 0/
Queue: Module-Install

People
Owner: Nobody in particular
Requestors: mschwern [...] cpan.org
Cc:
AdminCc:

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



Subject: Don't use ExtUtils::Install to copy into blib
Module::Install::Share uses MOD_INSTALL, the ExtUtils::Install macro, to copy files into blib. Don't do that, use CP. Why? ExtUtils::Install has hooks that can be toyed with directly to change where it installs things. Namely PERL_INSTALL_ROOT which acts kind of like DESTDIR. If an install tool sets PERL_INSTALL_ROOT you'll find your files ending up who knows where. The rpm build tool did this. For example, try running "make install" on SQL::Translator with PERL_INSTALL_ROOT set. You'll see the share dir winds up in /$PERL_INSTALL_ROOT/blib/...
Hi. Replaced MOD_INSTALL with CP and MKPATH in the trunk; will be fixed in the next release. Thanks. On 2010-1-25 Mon 15:26:53, MSCHWERN wrote: Show quoted text
> Module::Install::Share uses MOD_INSTALL, the ExtUtils::Install macro,
to Show quoted text
> copy files into blib. Don't do that, use CP. > > Why? ExtUtils::Install has hooks that can be toyed with directly to > change where it installs things. Namely PERL_INSTALL_ROOT which acts > kind of like DESTDIR. If an install tool sets PERL_INSTALL_ROOT
you'll Show quoted text
> find your files ending up who knows where. The rpm build tool did
this. Show quoted text
> > For example, try running "make install" on SQL::Translator with > PERL_INSTALL_ROOT set. You'll see the share dir winds up in > /$PERL_INSTALL_ROOT/blib/...
Hi. 0.95 is out. If you still have the problem, reopen this. Thanks.