If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Support multiple legacy checks

VERIFIED FIXED in mozilla1.3final

Status

SeaMonkey
Installer
P3
normal
VERIFIED FIXED
15 years ago
13 years ago

People

(Reporter: Samir Gehani, Assigned: Samir Gehani)

Tracking

Trunk
mozilla1.3final
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [adt2])

Attachments

(1 attachment)

3.40 KB, patch
Sean Su
: review+
(not reading, please use seth@sspitzer.org instead)
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

15 years ago
We need to support detection and alert of more than one binary filename since
Netscape 6 shipped with mozilla-bin but later versions shipped with
netscape-bin.  So during upgrade when we warn users (legacy check) to delete
their previous installation we need the ability to check for the existence of
_multiple_ binaries.
(Assignee)

Comment 1

15 years ago
I have a patch.
Assignee: dveditz → sgehani
Priority: -- → P3
Target Milestone: --- → mozilla1.3final
(Assignee)

Comment 2

15 years ago
Created attachment 113715 [details] [diff] [review]
Patch v1.0.

Updated

15 years ago
QA Contact: bugzilla → ktrina
(Assignee)

Comment 3

15 years ago
Comment on attachment 113715 [details] [diff] [review]
Patch v1.0.

Sean, please r.
Seth, please rs.

Note: the nsSetupTypeDlg.cpp patch is -w: the whitespace in my tree complies
with prevailing usage in the file.
Attachment #113715 - Flags: superreview?(sspitzer)
Attachment #113715 - Flags: review?(ssu)

Comment 4

15 years ago
Comment on attachment 113715 [details] [diff] [review]
Patch v1.0.

r=ssu
Attachment #113715 - Flags: review?(ssu) → review+
Comment on attachment 113715 [details] [diff] [review]
Patch v1.0.

rs=sspitzer
Attachment #113715 - Flags: superreview?(sspitzer) → superreview+

Updated

15 years ago
QA Contact: ktrina → gbush
Installer triage team: nsbeta1+/adt2
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt2]
(Assignee)

Comment 7

15 years ago
Landed on trunk.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Comment 8

15 years ago
verified 
both 6.x and 7.x previous installs are detected
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.