Bug #80257 for Win32-API: presence of cygpath needs to be check on module load

Thu Oct 18 01:15:46 2012 BULKDD [...] cpan.org - Ticket created

Subject:

presence of cygpath needs to be check on module load

On Cygwin builds, a backquote call is made to cygpath for DLL path conversion. If cygwin perl is started without cygwin being in the PATH, cygpath will not be found and the DLL path turns to empty string leading and fails, possibly with a bad $^E (I got "network connection lost" but that might be part of my Perl debugger). Add a test call to cygpath during the modules require phase. Die during module load if cygpath backquote test call (I'm thinking a `cygpath -V` and an index string search) didn't work. Also check to make sure " if ($^O eq 'cygwin' and $dll ne File::Basename::basename($dll)) {" is const folded away on VC Perl. This is a reminder for myself.

Thu Oct 18 11:36:03 2012 BULKDD [...] cpan.org - Correspondence added

Implemented.

Thu Oct 18 11:36:17 2012 BULKDD [...] cpan.org - Status changed from 'new' to 'resolved'

Thu Oct 18 12:01:40 2012 RURBAN [...] cpan.org - Correspondence added

On Thu Oct 18 01:15:46 2012, BULKDD wrote: Show quoted text

> On Cygwin builds, a backquote call is made to cygpath for DLL path > conversion. If cygwin perl is started without cygwin being in the

PATH, Show quoted text

> cygpath will not be found and the DLL path turns to empty string

leading Show quoted text

> and fails, possibly with a bad $^E (I got "network connection lost"

but Show quoted text

> that might be part of my Perl debugger). Add a test call to cygpath > during the modules require phase. Die during module load if cygpath > backquote test call (I'm thinking a `cygpath -V` and an index string > search) didn't work. Also check to make sure " if ($^O eq > 'cygwin' and $dll ne File::Basename::basename($dll)) {" is const

folded Show quoted text

> away on VC Perl. This is a reminder for myself.

A cygwin system cannot work at all if the path to cygwin\bin is not in the PATH. E.g. cygwin perl will not work. A lot of tests which reset or localize a PATH have special CYGWIN code to set the PATH to include \cygwin\bin. So it's a very esoteric issue you are fighting here, and really not worth checking. -- Reini Urban

Thu Oct 18 12:01:44 2012 The RT System itself - Status changed from 'resolved' to 'open'

Sat Oct 20 20:21:05 2012 BULKDD [...] cpan.org - Correspondence added

The cygpath changes are now available in Win32::API 0.73 on CPAN.

Sat Oct 20 20:21:07 2012 BULKDD [...] cpan.org - Status changed from 'open' to 'resolved'