Closed Bug 297147 Opened 19 years ago Closed 19 years ago

Update Windows registry after software update

Categories

(Toolkit :: Application Update, defect)

x86
Windows XP
defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla1.8final

People

(Reporter: darin.moz, Assigned: darin.moz)

References

Details

(Keywords: fixed1.8, Whiteboard: [eta 7/27])

Attachments

(1 file, 1 obsolete file)

Update Windows registry after software update

Installer builds of Firefox setup various registry keys that include the Firefox
version number.  We will need to update those keys after an update is applied to
Firefox.
Status: NEW → ASSIGNED
Flags: blocking1.8b4?
Flags: blocking-aviary1.1?
Target Milestone: --- → Firefox1.1
Need this for b4.
Flags: blocking1.8b4?
Flags: blocking1.8b4+
Flags: blocking-aviary1.1?
Depends on: 301521
Attached patch v1 patch (obsolete) — Splinter Review
bsmedberg:

This is the code for updating the windows registry that we spoke about.  I
intend to extend this code to actually write an install-wizard.log file so that
the uninstaller will know how to clean up the registry keys created by this
code.  That will be done as part of the patch for bug 301253.

As we also discussed, I'm interested in simply doing away with the registry
setting code done by the uninstaller in favor of letting this code handle all
registry updating on windows.  There's also an XXX comment in the code about
copying the latest Uninstall$appname.exe into C:\WINDOWS.  I will add that once
I figure out how to package the Uninstall$appname.exe in the generated MAR
files ;-)
Attachment #190483 - Flags: review?(benjamin)
Comment on attachment 190483 [details] [diff] [review]
v1 patch

This should not be a in a locale package... invent a new
chrome://branding/content/registryInfo.properties if you must.

Spelling nit: "heirarchy"

Otherwise it looks fine.
Attachment #190483 - Flags: review?(benjamin) → review-
> This should not be a in a locale package... invent a new
> chrome://branding/content/registryInfo.properties if you must.

Ok.  What jar file should it go into then?
Whiteboard: [eta 7/27]
Attached patch v2 patchSplinter Review
Okay, this patch uses the branding content package.
Attachment #190483 - Attachment is obsolete: true
Attachment #190658 - Flags: review?(benjamin)
Attachment #190658 - Flags: review?(benjamin)
Attachment #190658 - Flags: review+
Attachment #190658 - Flags: approval1.8b4+
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Keywords: fixed1.8
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: