Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the ExtUtils-Manifest CPAN distribution.

Report information
The Basics
Id:
39934
Status:
resolved
Priority:
Low/Low

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

BugTracker
Severity:
(no value)
Broken in:
(no value)
Fixed in:
(no value)



Subject: [PATCH] test nit with space in filename on VMS
Some versions of VMS with certain environment settings will successfully create a file with a space in the name but the actual file created has the space stripped out ('foo bar' becomes 'foobar'). That wreaks havoc with manifest.t. The easiest way to deal with that is to add a one- line sanity check that the file exists under the name we gave it, and that's what the attached patch does. This fixes a failure in blead that is also present in the 5.8.x branch soon-to-be 5.8.9.
Subject: euman_vms.patch
--- t/manifest.t;-0 Fri Sep 19 16:04:11 2008 +++ t/manifest.t Thu Oct 9 17:31:24 2008 @@ -35,8 +35,9 @@ sub add_file { 1 while unlink $file; # or else we'll get multiple versions on VMS open( T, '> '.$file) or return; print T $data; - ++$Files{$file}; close T; + return 0 unless -e $file; # exists under the name we gave it ? + ++$Files{$file}; } sub read_manifest {
Thanks very much - I've applied that in the svn sources, and will release a new CPAN version shortly.


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.