Closed Bug 477000 Opened 15 years ago Closed 15 years ago

Remove installer code that adds unused registry key values

Categories

(Firefox :: Installer, defect)

3.5 Branch
x86
Windows Vista
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3.6a1

People

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

Details

(Keywords: fixed1.9.1)

Attachments

(1 file)

At the very least no longer add
Mozilla\<App Name>\<version> (<locale>)\Main\Program Folder Path
which is not used and is currently set with an incorrect path
Also, Uninstall Log Folder
Summary: Remove adds of unused registry key values → Remove installer code that adds unused registry key values
Attached patch patch rev1Splinter Review
Hey Jim, we don't use these two registry key values for anything and the one for 'Program Folder Path' is being incorrectly anyways... I'd like to get rid of them. Thanks
Attachment #360872 - Flags: review?(jmathies)
Attachment #360872 - Flags: review?(jmathies) → review+
Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/48e371648b83
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.2a1
Comment on attachment 360872 [details] [diff] [review]
patch rev1

Drivers, I'd like to get this for Firefox 3.1. This patch removes code that sets a couple of registry keys we don't use and one of the keys is written incorrectly.
Attachment #360872 - Flags: approval1.9.1?
Comment on attachment 360872 [details] [diff] [review]
patch rev1

a191=beltzner
Attachment #360872 - Flags: approval1.9.1? → approval1.9.1+
To verify this run the installer and check that the following registry keys don't exist where
${BrandFullNameInternal} = Minefield or Shiretoko
${AppVersion} = the version of the app e.g. 3.1b3pre or 3.2a1pre
${AB_CD} = the locale e.g. en-US

Path = Software\Mozilla\${BrandFullNameInternal}\${AppVersion} (${AB_CD})\Main"
Value name = "Program Folder Path"
 
Path = "Software\Mozilla\${BrandFullNameInternal}\${AppVersion} (${AB_CD})\Uninstall"
Value name = "Uninstall Log Folder"
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: