131521 - Use of uninitialized value in split at /usr/lib/perl5/5.6.1/Cwd.pm line 103.

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect, P1)

Description

[Bradley Baetz (:bbaetz)]

perl 5.6.1 has shipped with a Cwd.pm which gives a warning if $ENV{PATH} doesn't
exist (see the above url). This is apparently fixed in bleedperl, but snice I
get one of these (+ detained explanation) in the logs each time processmail is
called, I figure we want to stop it.

The patch is just to set $::ENV{'PATH'} = ''; after deleting it in globals.pl (I
can't produce a diff easily since I have too many other changes in my globals.pl)

I think we want to do this in 2.16 to stop warnings, and I can't think of
anything which will break (Cwd.pm is part of perl, not a separate modules, so we
can't just tell people to upgrade), so I'll set the milestone.

Comment 1

[Bradley Baetz (:bbaetz)]

Attachment: patch

Comment 2

[David D. Kilzer (ddk)]

Comment on attachment 74584 [details] [diff] [review]
patch

r=ddk

Nit: Add a comment above the line to note why we are doing this ("bug" in
Cwd.pm in Perl 5.6.1).

Comment 3

[Bradley Baetz (:bbaetz)]

Attachment: v2

Comment 4

[Dave Miller [:justdave]]

Comment on attachment 75712 [details] [diff] [review]
v2

r= justdave

Comment 5

[Bradley Baetz (:bbaetz)]

Checked in.