Closed Bug 54372 Opened 24 years ago Closed 24 years ago

installer creates C:\program files\Netscape\Netscape 6\install_wizard.log even if I point it to install somewhere else

Categories

(SeaMonkey :: Installer, defect, P3)

Product:
SeaMonkey
Component:
Installer
Platform:
x86
Windows NT
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: sspitzer, Assigned: ssu0262)

References

Details

(Whiteboard: [rtm-])

Attachments

(2 files)

patch to fix this bug
(deleted), patch
Details | Diff | Splinter Review
2nd patch.
(deleted), patch
Details | Diff | Splinter Review
I use the installer (N6Setup.exe) but I point it to install on D:\bits when I finished, I noticed that it created C:\Program Files\Netscape\Netscape 6\install_wizard.log
I see it.
Status: NEW → ASSIGNED
Attached patch patch to fix this bug (deleted) — — Splinter Review
adding rtm keyword.
Keywords: rtm
Whiteboard: [rtm+ need info]
Sean, please get this fix super-reviewed if you still think it should go into rtm.
Whiteboard: [rtm+ need info] → [rtm+ need review]
I am trying. I was busy with bugscape 2778, and didn't want to have two bug fixes confusing me. Samir reviewed this patch and found a problem. I will be requesting a sr= for this bug on Friday after I fix the problem.
attaching 2nd patch. waiting on r= from Samir.
Attached patch 2nd patch. (deleted) — — Splinter Review
r=dveditz for updated patch
sr=mscott
removing "need review" from status whiteboard since r=dveditz and sr=mscott
Whiteboard: [rtm+ need review] → [rtm+]
PDT marking [rtm-] because the bug seems really minor and the fix is bigger than a breadbox.
Whiteboard: [rtm+] → [rtm-]
BTW, please land on trunk or hold the patch until after seamonkey
fix applied to only the trunk. leaving bug open until fix is also applied to branch.
You'll have to remove rtm- if you want to get this back on PDT's radar if this is going to have any chance of being fixed in the branch...
I think that pdt meant it's not going on the branch until after we ship, at least that's the way I interpreted it. So removing rtm- will just make them put it right back.
But there won't be a branch after we ship, other than perhaps a firedrill 6.01 which would only have the firedrill fix. Unless you want to appeal the pdt's rtm- you might as well resolve the bug.
*** Bug 55742 has been marked as a duplicate of this bug. ***
marking this fixed since N6 has shipped and this was already fixed in the trunk.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
changing QA and vfy N6 fixed in trunk
Status: RESOLVED → VERIFIED
QA Contact: gemal → gbush
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: