162217 - ./checksetup.pl: Warning after pulldown menu support in defparams.pl

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, defect, P1)

Description

[Tobias Burnus]

After bug 160410 has been checked in I get this warning when running
./checksetup.pl:
Checking user setup ...
Use of uninitialized value in string eq at globals.pl line 1443 (#1)
    (W uninitialized) An undefined value was used as if it were already
    defined.  It was interpreted as a "" or a 0, but maybe it was a mistake.
    To suppress this warning assign a defined value to your variables.
    
    To help you figure out what was undefined, perl tells you what operation
    you used the undefined value in.  Note, however, that perl optimizes your
    program and the operation displayed in the warning may not necessarily
    appear literally in your program.  For example, "that $foo" is
    usually optimized into "that " . $foo, and the warning will refer to
    the concatenation (.) operator, even though there is no . in your
    program.
    
[Sun Aug 11 23:45:49 2002] checksetup.pl: Use of uninitialized value in string
eq at globals.pl line 1443.
[Sun Aug 11 23:45:49 2002] checksetup.pl: Use of uninitialized value in string
eq at globals.pl line 1443.

Comment 1

[J. Paul Reed [:preed]]

Yes, I'm aware of this, and yes it's in the process of being fixed.

Please confirm the version of perl you're using; this warning does not appear on
perl 5.6.x, making tracking it down a pain.

Comment 2

[Tobias Burnus]

> Please confirm the version of perl you're using
This is perl, v5.6.1 built for i386-linux
> this warning does not appear on perl 5.6.x, making tracking it down a pain.
Hmm.

Comment 3

[J. Paul Reed [:preed]]

Hrm.

Ok... I tweaked the testing suite and got it to reproduce those errors... which
is a good thing(tm).

Comment 4

[J. Paul Reed [:preed]]

Fix for this checked in:

Checking in globals.pl;
/cvsroot/mozilla/webtools/bugzilla/globals.pl,v  <--  globals.pl
new revision: 1.188; previous revision: 1.187
done

and

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=Bugzilla&branch=HEAD&branchtype=match&dir=&file=globals.pl&filetype=match&who=preed%25sigkill.com&whotype=match&sortby=Date&hours=2&date=explicit&mindate=08%2F10%2F2002&maxdate=08%2F12%2F2002&cvsroot=%2Fcvsroot

Waiting for tinderbox to go green before I mark fixed.

Comment 5

[J. Paul Reed [:preed]]

Bugzilla tip tinderbox went green as of 15:08 today.

Closing.

And my apologies... :-/