Skip Menu |

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

Report information
The Basics
Id: 35190
Status: resolved
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: Nobody in particular
Requestors: imacat [...] mail.imacat.idv.tw
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 6.44
Fixed in: (no value)



Subject: Use of uninitialized value in length at /usr/lib/perl5/5.8.8/ExtUtils/Command/MM.pm line 134
Dear Michael G Schwern, Hi. This is imacat from Taiwan. I encounter numerous warnings when I install mod_perl 2. They all says: ... Use of uninitialized value $options{"perm_rw"} in length at /tmp/perl-mod_perl2/5.10.0/lib/5.10.0/ExtUtils/Command/MM.pm line 134. ... I made a simple patch against ExtUtils::Command::MM in ExtUtils-MakeMaker-6.44 that can remove this warning. Hope that this helps. Please tell me if you need any more information, or if I could be of any help. Thank you.
Subject: ExtUtils-MakeMaker-6.44-perm_rw.diff
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 diff -u -r ExtUtils-MakeMaker-6.44.orig/lib/ExtUtils/Command/MM.pm ExtUtils-MakeMaker-6.44/lib/ExtUtils/Command/MM.pm - --- ExtUtils-MakeMaker-6.44.orig/lib/ExtUtils/Command/MM.pm 2008-02-29 08:06:28.000000000 +0800 +++ ExtUtils-MakeMaker-6.44/lib/ExtUtils/Command/MM.pm 2008-04-19 00:16:51.000000000 +0800 @@ -131,7 +131,7 @@ $parser->parse_from_file($pod, $man) or do { warn("Could not install $man\n"); next }; - - if (length $options{perm_rw}) { + if (exists $options{perm_rw}) { chmod(oct($options{perm_rw}), $man) or do { warn("chmod $options{perm_rw} $man: $!\n"); next }; } -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iEYEARECAAYFAkgIyZcACgkQi9gubzC5S1zLGACgp+xcRA3wEw1jEbqxjrAI2cae XKYAnilV07lKhNL81uAUaFS/iBKnu8+U =Z5RC -----END PGP SIGNATURE-----
Thanks for the catch. I've also noticed that perm_rw should be set perm_rw=i so that a value is required. If you're feeling ambitious, ExtUtils::Command::MM::pod2man has no direct tests...
Finally got around to applying this, thanks.