Closed Bug 486780 Opened 15 years ago Closed 15 years ago

EndUnless used in installer where EndIf should be used

Categories

(Firefox :: Installer, defect)

x86
Windows XP
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Firefox 3.5b4

People

(Reporter: mcsmurf, Assigned: robert.strong.bugs)

Details

(Keywords: fixed1.9.1)

Attachments

(1 file)

See http://hg.mozilla.org/mozilla-central/annotate/6f3c2171bbb2/browser/installer/windows/nsis/installer.nsi#l471, EndUnless does not match the {If} ... tag in line 465. Looks like NSIS seems to ignore/correct this?
Attached patch patchSplinter Review
Thanks for finding / reporting this Frank.
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
Attachment #372734 - Flags: review?(jmathies)
Attachment #372734 - Flags: review?(jmathies) → review+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/e00231915b17
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment on attachment 372734 [details] [diff] [review]
patch

Drivers, this is a very simple fix I'd like for 1.9.1 that has no risk.
Attachment #372734 - Flags: approval1.9.1?
Attachment #372734 - Flags: approval1.9.1? → approval1.9.1+
Pushed to mozilla-1.9.1
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/02a79ed478dd
Keywords: fixed1.9.1
Target Milestone: --- → Firefox 3.5b4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: