Closed
Bug 133210
Opened 24 years ago
Closed 24 years ago
checksetup uses $::params
Categories
(Bugzilla :: Installation & Upgrading, defect, P3)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: jayvdb, Assigned: jayvdb)
Details
Attachments
(1 file)
790 bytes,
patch
|
bbaetz
:
review+
zach
:
review+
|
Details | Diff | Splinter Review |
In bug 96675 (R1.16 blocker), the patch uses $::params rather than $::param.
Assignee | ||
Comment 1•24 years ago
|
||
![]() |
||
Comment 2•24 years ago
|
||
Comment on attachment 75921 [details] [diff] [review]
patch
r=bbaetz
Attachment #75921 -
Flags: review+
![]() |
||
Comment 3•24 years ago
|
||
I guess that now we test, we should do this for 2.16
![]() |
||
Comment 4•24 years ago
|
||
Comment on attachment 75921 [details] [diff] [review]
patch
r2=zach
Attachment #75921 -
Flags: review+
![]() |
||
Comment 5•24 years ago
|
||
I note that we die "" if the email is invalid, which is ugly - I'll file another
bug on that one.
-> patch author....
Assignee: zach → zeroJ
![]() |
||
Comment 6•24 years ago
|
||
... and checked in
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•