Skip to content

Commit

Permalink
RT 74444 - install_lib.pl emits warnings when installperl is run with…
Browse files Browse the repository at this point in the history
… destdir

running the following produces 7 warnings like this on my system:
'LD_LIBRARY_PATH=/usr/home/build/perl-5.12.0  ./perl installperl --destdir=/var/local/tmp/perl-root'

Use of uninitialized value $dev2 in numeric eq (==) at ./install_lib.pl line 123.

replacing == with ~~ (the smart match operator) seems to be the best DWIM fix.
  • Loading branch information
toddr authored and rafl committed Sep 1, 2010
1 parent 2273b9b commit 4c3ca3f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion install_lib.pl
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ sub samepath {
my($dev1, $ino1, $dev2, $ino2); my($dev1, $ino1, $dev2, $ino2);
($dev1, $ino1) = stat($p1); ($dev1, $ino1) = stat($p1);
($dev2, $ino2) = stat($p2); ($dev2, $ino2) = stat($p2);
($dev1 == $dev2 && $ino1 == $ino2); ($dev1 ~~ $dev2 && $ino1 ~~ $ino2);
} }
else { else {
1; 1;
Expand Down

0 comments on commit 4c3ca3f

Please sign in to comment.