Bug #20528 for Module-Build: Module-Build installs into wrong location with bleadperl

Tue Jul 18 08:49:06 2006 shay [...] cpan.org - Ticket created

When manually installing (actually updating) Module-Build-0.2804 into my bleadperl installation (at patchlevel 28556), I found that it got installed into the site/lib/ tree, rather than the lib/ tree. So, of course, the version actually being *used* subsequently isn't changed because lib/ comes before site/lib/ in @INC. I did the install with the commands: perl Build.PL perl Build perl Build test perl Build install I'm running on Windows XP, using VC++ 6.0. Does the Build.PL need to play some games with installdirs now that Module-Build is in the core (for bleadperl anyway)? For example, Devel-PPPort has been in the core since 5.7.3 and contains this in its Makefile.PL: if ($ENV{'PERL_CORE'}) { # Pods will be built by installman. push @moreopts, MAN3PODS => {}; push @clean, 'PPPort.pm'; } else { # Devel::PPPort is in the core since 5.7.3 push @moreopts, INSTALLDIRS => ($] >= 5.007003 ? 'perl' : 'site'); } The attached patch does something similar for Module-Build, but it still isn't quite correct. The output below shows what happened: Installing C:\perl5\man\man1\config_data.1 Installing C:\perl5\html\bin\config_data.html Installing C:\perl5\lib\Module\Build.pm Skipping C:\perl5\lib\Module\Build\API.pod (unchanged) Skipping C:\perl5\lib\Module\Build\Authoring.pod (unchanged) Installing C:\perl5\lib\Module\Build\Base.pm Skipping C:\perl5\lib\Module\Build\Compat.pm (unchanged) Installing C:\perl5\lib\Module\Build\ConfigData.pm Skipping C:\perl5\lib\Module\Build\Cookbook.pm (unchanged) Installing C:\perl5\lib\Module\Build\ModuleInfo.pm Skipping C:\perl5\lib\Module\Build\Notes.pm (unchanged) Skipping C:\perl5\lib\Module\Build\PodParser.pm (unchanged) Skipping C:\perl5\lib\Module\Build\PPMMaker.pm (unchanged) Installing C:\perl5\lib\Module\Build\Version.pm Skipping C:\perl5\lib\Module\Build\YAML.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\aix.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\Amiga.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\cygwin.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\darwin.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\Default.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\EBCDIC.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\MacOS.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\MPEiX.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\os2.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\RiscOS.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\Unix.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\VMS.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\VOS.pm (unchanged) Skipping C:\perl5\lib\Module\Build\Platform\Windows.pm (unchanged) Installing C:\perl5\man\man3\Module.Build.3 Installing C:\perl5\man\man3\Module.Build.API.3 Installing C:\perl5\man\man3\Module.Build.Authoring.3 Installing C:\perl5\man\man3\Module.Build.Base.3 Installing C:\perl5\man\man3\Module.Build.Compat.3 Installing C:\perl5\man\man3\Module.Build.ConfigData.3 Installing C:\perl5\man\man3\Module.Build.Cookbook.3 Installing C:\perl5\man\man3\Module.Build.ModuleInfo.3 Installing C:\perl5\man\man3\Module.Build.Notes.3 Installing C:\perl5\man\man3\Module.Build.Platform.aix.3 Installing C:\perl5\man\man3\Module.Build.Platform.Amiga.3 Installing C:\perl5\man\man3\Module.Build.Platform.cygwin.3 Installing C:\perl5\man\man3\Module.Build.Platform.darwin.3 Installing C:\perl5\man\man3\Module.Build.Platform.Default.3 Installing C:\perl5\man\man3\Module.Build.Platform.EBCDIC.3 Installing C:\perl5\man\man3\Module.Build.Platform.MacOS.3 Installing C:\perl5\man\man3\Module.Build.Platform.MPEiX.3 Installing C:\perl5\man\man3\Module.Build.Platform.os2.3 Installing C:\perl5\man\man3\Module.Build.Platform.RiscOS.3 Installing C:\perl5\man\man3\Module.Build.Platform.Unix.3 Installing C:\perl5\man\man3\Module.Build.Platform.VMS.3 Installing C:\perl5\man\man3\Module.Build.Platform.VOS.3 Installing C:\perl5\man\man3\Module.Build.Platform.Windows.3 Installing C:\perl5\man\man3\Module.Build.PPMMaker.3 Installing C:\perl5\man\man3\Module.Build.YAML.3 Installing C:\perl5\html\lib\Module\Build.html Installing C:\perl5\html\lib\Module\Build\API.html Installing C:\perl5\html\lib\Module\Build\Authoring.html Installing C:\perl5\html\lib\Module\Build\Base.html Installing C:\perl5\html\lib\Module\Build\Compat.html Installing C:\perl5\html\lib\Module\Build\ConfigData.html Installing C:\perl5\html\lib\Module\Build\Cookbook.html Installing C:\perl5\html\lib\Module\Build\ModuleInfo.html Installing C:\perl5\html\lib\Module\Build\Notes.html Installing C:\perl5\html\lib\Module\Build\PPMMaker.html Installing C:\perl5\html\lib\Module\Build\YAML.html Installing C:\perl5\html\lib\Module\Build\Platform\aix.html Installing C:\perl5\html\lib\Module\Build\Platform\Amiga.html Installing C:\perl5\html\lib\Module\Build\Platform\cygwin.html Installing C:\perl5\html\lib\Module\Build\Platform\darwin.html Installing C:\perl5\html\lib\Module\Build\Platform\Default.html Installing C:\perl5\html\lib\Module\Build\Platform\EBCDIC.html Installing C:\perl5\html\lib\Module\Build\Platform\MacOS.html Installing C:\perl5\html\lib\Module\Build\Platform\MPEiX.html Installing C:\perl5\html\lib\Module\Build\Platform\os2.html Installing C:\perl5\html\lib\Module\Build\Platform\RiscOS.html Installing C:\perl5\html\lib\Module\Build\Platform\Unix.html Installing C:\perl5\html\lib\Module\Build\Platform\VMS.html Installing C:\perl5\html\lib\Module\Build\Platform\VOS.html Installing C:\perl5\html\lib\Module\Build\Platform\Windows.html Installing C:\perl5\html\site\lib\Module\Build.html Installing C:\perl5\html\site\lib\Module\Build\API.html Installing C:\perl5\html\site\lib\Module\Build\Authoring.html Installing C:\perl5\html\site\lib\Module\Build\Base.html Installing C:\perl5\html\site\lib\Module\Build\Compat.html Installing C:\perl5\html\site\lib\Module\Build\ConfigData.html Installing C:\perl5\html\site\lib\Module\Build\Cookbook.html Installing C:\perl5\html\site\lib\Module\Build\ModuleInfo.html Installing C:\perl5\html\site\lib\Module\Build\Notes.html Installing C:\perl5\html\site\lib\Module\Build\PPMMaker.html Installing C:\perl5\html\site\lib\Module\Build\YAML.html Installing C:\perl5\html\site\lib\Module\Build\Platform\aix.html Installing C:\perl5\html\site\lib\Module\Build\Platform\Amiga.html Installing C:\perl5\html\site\lib\Module\Build\Platform\cygwin.html Installing C:\perl5\html\site\lib\Module\Build\Platform\darwin.html Installing C:\perl5\html\site\lib\Module\Build\Platform\Default.html Installing C:\perl5\html\site\lib\Module\Build\Platform\EBCDIC.html Installing C:\perl5\html\site\lib\Module\Build\Platform\MacOS.html Installing C:\perl5\html\site\lib\Module\Build\Platform\MPEiX.html Installing C:\perl5\html\site\lib\Module\Build\Platform\os2.html Installing C:\perl5\html\site\lib\Module\Build\Platform\RiscOS.html Installing C:\perl5\html\site\lib\Module\Build\Platform\Unix.html Installing C:\perl5\html\site\lib\Module\Build\Platform\VMS.html Installing C:\perl5\html\site\lib\Module\Build\Platform\VOS.html Installing C:\perl5\html\site\lib\Module\Build\Platform\Windows.html Installing C:\perl5\bin\config_data Installing C:\perl5\bin\config_data.bat Writing C:\perl5\lib\auto\Module\Build\.packlist So the installation of the code is OK now, but two sets of HTML are being installed: one in lib/ and one in site/lib/.

Subject:

patch.txt

--- Build.PL.orig 2006-07-16 22:50:44.000000000 +0100 +++ Build.PL 2006-07-18 13:40:22.730426400 +0100 @@ -81,6 +81,8 @@ repository => 'http://svn.perl.org/modules/Module-Build/' } }, + + installdirs => ($] >= 5.009004 ? 'core' : 'site'), ); $build->create_build_script;

Tue Jul 18 10:02:42 2006 shay [...] cpan.org - Status changed from 'new' to 'open'

Tue Jul 18 10:04:26 2006 shay [...] cpan.org - Correspondence added

Subject:

Module-Build installs into wrong location with bleadperl

Oops! Didn't mean to change status to "open". Resetting it to "new". I was actually just trying to fill in a Subject, which I forgot originally!

Tue Jul 18 10:04:27 2006 shay [...] cpan.org - Status changed from 'open' to 'new'

Tue Jul 18 10:07:59 2006 shay [...] cpan.org - Correspondence added

Subject:

Module-Build installs into wrong location with bleadperl

I give up, filling in the Subject line when updating the ticket just doesn't seem to work now.

Sun Jul 30 15:36:59 2006 kwilliams [...] cpan.org - Subject changed from (no value) to 'Module-Build installs into wrong location with bleadperl'

Sun Oct 07 14:18:12 2007 EWILHELM [...] cpan.org - Correspondence added

On Tue Jul 18 08:49:06 2006, SHAY wrote: Show quoted text

> When manually installing (actually updating) Module-Build-0.2804 > into my bleadperl installation (at patchlevel 28556), I found that > it got installed into the site/lib/ tree, rather than the lib/ tree.

Hi Steve, I'm trying to dust-off the rt queue and it looks like this one may have been resolved in the last year. Can you please try with 0.2808 from CPAN and/or the current svn and let us know if we still have this bug? https://svn.perl.org/modules/Module-Build/trunk Thanks, Eric

Sun Oct 07 14:18:14 2007 The RT System itself - Status changed from 'new' to 'open'

Sun Oct 07 14:18:16 2007 EWILHELM [...] cpan.org - Status changed from 'open' to 'stalled'

Sun Oct 07 14:19:06 2007 EWILHELM [...] cpan.org - Reminder 'probably resolved' added

Mon Oct 08 07:31:27 2007 shay [...] cpan.org - Correspondence added

From:

SHAY [...] cpan.org

On Sun Oct 07 14:18:12 2007, EWILHELM wrote: Show quoted text

> On Tue Jul 18 08:49:06 2006, SHAY wrote:

> > When manually installing (actually updating) Module-Build-0.2804 > > into my bleadperl installation (at patchlevel 28556), I found that > > it got installed into the site/lib/ tree, rather than the lib/ tree.

> > Hi Steve, > > I'm trying to dust-off the rt queue and it looks like this one may > have been resolved in the last year. Can you please try with 0.2808 > from CPAN and/or the current svn and let us know if we still have this > bug?

I tried both 0.2808 and current svn, and I'm afraid that they both still have the bug. I installed them each into a fresh bleadperl (@32066) and they both installed into site/lib rather than lib.

Mon Oct 08 07:31:35 2007 The RT System itself - Status changed from 'stalled' to 'open'

Thu Oct 25 12:18:04 2007 EWILHELM [...] cpan.org - Reminder 'probably resolved' completed

Thu Oct 25 13:10:50 2007 EWILHELM [...] cpan.org - Correspondence added

Fixed in svn r10118 (look for v0.2809 soon.) Thanks, Eric

Thu Oct 25 13:10:54 2007 EWILHELM [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Jan 10 09:19:51 2008 shay [...] cpan.org - Correspondence added

On Thu Oct 25 13:10:50 2007, EWILHELM wrote: Show quoted text

> Fixed in svn r10118 (look for v0.2809 soon.) >

Sorry, but it doesn't look quite fixed to me :-( I just tried installing r10504 into perl-5.10.0 and I find that the code (*.pm) is correctly installed under lib/ but the HTML documentation gets installed twice--once under html/lib and once more under html/site/lib. I referred to exactly this problem having occurred with the attempted fix that I suggested in my original report.

Thu Jan 10 09:19:55 2008 The RT System itself - Status changed from 'resolved' to 'open'

Mon Jan 12 22:38:10 2009 EWILHELM [...] cpan.org - Correspondence added

On Thu Jan 10 09:19:51 2008, SHAY wrote: Show quoted text

> On Thu Oct 25 13:10:50 2007, EWILHELM wrote:

>> Fixed in svn r10118

> Sorry, but it doesn't look quite fixed to me :-( > > ... the HTML documentation gets installed twice

Hi Steve, I seem to not have binhtml and libhtml in my Config.pm on debian, so I'm having trouble tracking this down. Could you please append these lines to the Build.PL and send me the output? require Module::Build::YAML; warn Module::Build::YAML::Dump($build->install_map); Thanks, Eric

Mon Jan 12 22:38:30 2009 EWILHELM [...] cpan.org - Status changed from 'open' to 'stalled'

Mon Jan 12 22:38:30 2009 EWILHELM [...] cpan.org - Severity Important changed to Normal

Tue Jan 13 04:35:33 2009 SteveHay [...] planit.com - Correspondence added

Subject:	RE: [rt.cpan.org #20528] Module-Build installs into wrong location with bleadperl
Date:	Tue, 13 Jan 2009 09:34:13 -0000
To:	<bug-Module-Build [...] rt.cpan.org>
From:	"Steve Hay" <SteveHay [...] planit.com>

Message body is not shown because it is too large.

Download dotpacklist
application/octet-stream 3.5k

Message body not shown because it is not plain text.

Message body is not shown because sender requested not to inline it.

Tue Jan 13 04:35:35 2009 The RT System itself - Status changed from 'stalled' to 'open'

Tue Jan 13 12:23:12 2009 EWILHELM [...] cpan.org - Status changed from 'open' to 'resolved'