Skip Menu |

This queue is for tickets about the local-lib CPAN distribution.

Report information
The Basics
Id: 47390
Status: resolved
Priority: 0/
Queue: local-lib

People
Owner: Nobody in particular
Requestors: emazep [...] gmail.com
Cc:
AdminCc:

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



Subject: local::lib assumes MSWin32 even under Cygwin
Checking for $ENV{COMSPEC} defined-ness is not enough to tell MSWin32 from Cygwin, since they both have that variable defined. Thus local::lib wrongly assumes to be under MSWin32 even under Cygwin, causing it to use the "set" command (which is unknown under any Cygwin shell) instead of "export" or "setenv". It's a regression which unfortunately makes local::lib completely ineffective under Cygwin now. Patch attached. Thank you for your great work! -Emanuele Zeppieri
Subject: lib.pm.patch
--- lib.pm.orig 2009-06-27 09:23:20.652900000 +0200 +++ lib.pm 2009-06-27 10:17:57.355900000 +0200 @@ -272,8 +272,8 @@ } }; - # Win32 uses this variable. - if (defined $ENV{'COMSPEC'}) { + # Both Win32 and Cygwin have $ENV{COMSPEC} set. + if (defined $ENV{'COMSPEC'} && $^O ne 'cygwin') { my @shell_bin_path_parts = File::Spec->splitpath($ENV{'COMSPEC'}); $shellbin = $shell_bin_path_parts[-1]; $shelltype = do {
This was fixed in the release of 1.4.5 but I forgot to close the ticket because I suck.