Bug #40262 for Module-Build: LIB override mangled when building v0.30

Wed Oct 22 07:22:58 2008 andrew.benham [...] thus.net - Ticket created

Subject:	LIB override mangled when building v0.30
Date:	Wed, 22 Oct 2008 12:22:25 +0100
To:	bug-Module-Build [...] rt.cpan.org
From:	Andrew Benham <andrew.benham [...] thus.net>

Attempting to build Module::Build v0.30 on Linux x86, Solaris sparc, and Solaris x86. Our LIB override is mangled. Firstly, v0.2808 did the right thing: $ perl Makefile.PL PREFIX=/opt/thus LIB=/opt/thus/share/libperl # running Build.PL --prefix /opt/thus --install_path lib=/opt/thus/share/libperl /usr/bin/perl Build.PL --prefix /opt/thus --install_path lib=/opt/thus/share/libperl ... $ make install DESTDIR=/tmp/builder /usr/bin/perl Build --makefile_env_macros 1 install Installing /tmp/builder/opt/thus/share/man/man1/config_data.1 Installing /tmp/builder/opt/thus/share/libperl/Module/Build.pm Installing /tmp/builder/opt/thus/share/libperl/Module/Build/Config.pm ... v0.30 is different, rather than using what we specified for 'LIB', it makes its own directory up: $ perl Makefile.PL PREFIX=/opt/thus LIB=/opt/thus/share/libperl # running Build.PL --prefix /opt/thus --config installsitelib=/opt/thus/share/libperl --config installprivlib=/opt/thus/share/libperl --config installarchlib=/opt/thus/share/libperl/i386-linux-thread-multi --config installsitearch=/opt/thus/share/libperl/i386-linux-thread-multi ... $ make install DESTDIR=/tmp/builder /usr/bin/perl Build --makefile_env_macros 1 install Installing /tmp/builder/opt/thus/share/man/man1/config_data.1 Installing /tmp/builder/opt/thus/lib/perl5/site_perl/Module/Build.pm Installing /tmp/builder/opt/thus/lib/perl5/site_perl/Module/Build/Config.pm ... -- Andrew Benham Demon@Thus@Cable&Wireless andrew.benham@thus.net Finchley, London N3 2QQ, U.K. Tel: 020 8495 6343 Fax: 020 8495 6037

Fri Oct 24 03:23:59 2008 schwern [...] pobox.com - Correspondence added

Subject:	Re: [rt.cpan.org #40262] LIB override mangled when building v0.30
Date:	Fri, 24 Oct 2008 00:23:02 -0700
To:	bug-Module-Build [...] rt.cpan.org
From:	Michael G Schwern <schwern [...] pobox.com>

This is probably a side effect of https://svn.perl.org/modules/Module-Build/trunk@11816 which changed the way LIB is implemented to fix INSTALLDIRS and INSTALL*LIB emulation. I'll look into it. -- If at first you don't succeed--you fail. -- "Portal" demo

Fri Oct 24 03:24:02 2008 The RT System itself - Status changed from 'new' to 'open'

Thu Nov 06 09:02:34 2008 TONVOON [...] cpan.org - Correspondence added

On Fri Oct 24 03:23:59 2008, schwern@pobox.com wrote: Show quoted text

> This is probably a side effect of > https://svn.perl.org/modules/Module-Build/trunk@11816 > which changed the way LIB is implemented to fix INSTALLDIRS and INSTALL*LIB > emulation. > > I'll look into it. > >

I just wanted to mention that this will affect installs of perl modules from CPAN. It seems to only affect modules that call M::B::Compat->write_makefile. A small list of affected modules: RRD::Simple LWPx::TimedHTTP Data::Serializer Module::Build To recreate, download RRD-Simple-1.44.tar.gz, expand and run: PERL5LIB={point to M::B 0.2808} perl Makefile.PL PREFIX=/opt/perl INSTALLDIRS=site LIB=/opt/perl/lib make ./Build fakeinstall Will see it installs into /opt/perl/lib/RRD/Simple.pm Now with: PERL5LIB={point to M::B 0.30} perl Makefile.PL PREFIX=/opt/perl INSTALLDIRS=site LIB=/opt/perl/lib make ./Build fakeinstall This will show it installs into: /opt/perl/lib/perl5/site_perl/RRD/Simple.pm Ton

Thu Nov 06 11:47:28 2008 kwilliams [...] cpan.org - Correspondence added

Is this an issue with the way Makefile.PL is translating 'make' arguments to M::B arguments? Or with the way those are interpreted after they reach M::B?

Thu Dec 11 14:31:40 2008 EWILHELM [...] cpan.org - Severity Normal added

Tue Jan 13 02:09:20 2009 EWILHELM [...] cpan.org - Correspondence added

On Fri Oct 24 03:23:59 2008, schwern@pobox.com wrote: Show quoted text

> This is probably a side effect of > https://svn.perl.org/modules/Module-Build/trunk@11816 > > which changed the way LIB is implemented to fix INSTALLDIRS and > INSTALL*LIB emulation.

Hi Andrew, Could you please try the code from r11815 to verify whether Schwern's guess is correct? svn co -r 11815 https://svn.perl.org/modules/Module-Build/trunk module-build Thanks, Eric

Tue Jan 13 02:10:23 2009 EWILHELM [...] cpan.org - Status changed from 'open' to 'stalled'

Wed Jan 14 09:25:30 2009 andrew.benham [...] thus.net - Correspondence added

Subject:	Re: [rt.cpan.org #40262] LIB override mangled when building v0.30
Date:	Wed, 14 Jan 2009 14:25:06 +0000
To:	bug-Module-Build [...] rt.cpan.org
From:	Andrew Benham <andrew.benham [...] thus.net>

Eric Wilhelm via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=40262 > > > On Fri Oct 24 03:23:59 2008, schwern@pobox.com wrote:

>> This is probably a side effect of >> https://svn.perl.org/modules/Module-Build/trunk@11816 >> >> which changed the way LIB is implemented to fix INSTALLDIRS and >> INSTALL*LIB emulation.

> > Hi Andrew, > > Could you please try the code from r11815 to verify whether Schwern's > guess is correct? > > svn co -r 11815 https://svn.perl.org/modules/Module-Build/trunk > module-build

That code from Subversion is good - it does not exhibit the problem. -- Andrew Benham Demon@Thus@Cable&Wireless andrew.benham@thus.net Finchley, London N3 2QQ, U.K. Tel: 020 8495 6343 Fax: 020 8495 6037

Wed Jan 14 09:25:30 2009 The RT System itself - Status changed from 'stalled' to 'open'

Wed Jan 14 12:34:41 2009 EWILHELM [...] cpan.org - Status changed from 'open' to 'resolved'