Closed Bug 192150 Opened 22 years ago Closed 21 years ago

Support multiple legacy checks

Categories

(SeaMonkey :: Installer, defect, P3)

Product:
SeaMonkey
Component:
Installer
Platform:
x86
Linux
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.3final

People

(Reporter: samir_bugzilla, Assigned: samir_bugzilla)

Details

(Whiteboard: [adt2])

Attachments

(1 file)

Patch v1.0.
3.40 KB, patch
ssu0262
: review+
sspitzer
: superreview+
Details | Diff | Splinter Review 
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.
I have a patch.
Assignee: dveditz → sgehani
Priority: -- → P3
Target Milestone: --- → mozilla1.3final
Attached patch Patch v1.0. — — Splinter Review 

    
  
QA Contact: bugzilla → ktrina

    
    

    
  
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 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+

    
  
QA Contact: ktrina → gbush

    
    

    
  
Installer triage team: nsbeta1+/adt2
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt2]

    
    

    
  
Landed on trunk.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED

    
    

    
  
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.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: