Closed
Bug 297147
Opened 19 years ago
Closed 19 years ago
Update Windows registry after software update
Categories
(Toolkit :: Application Update, defect)
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)
30.74 KB,
patch
|
benjamin
:
review+
benjamin
:
approval1.8b4+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Flags: blocking1.8b4?
Flags: blocking-aviary1.1?
Target Milestone: --- → Firefox1.1
Comment 1•19 years ago
|
||
Need this for b4.
Flags: blocking1.8b4?
Flags: blocking1.8b4+
Flags: blocking-aviary1.1?
Assignee | ||
Comment 2•19 years ago
|
||
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 ;-)
Assignee | ||
Updated•19 years ago
|
Attachment #190483 -
Flags: review?(benjamin)
Comment 3•19 years ago
|
||
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-
Assignee | ||
Comment 4•19 years ago
|
||
> 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?
Comment 5•19 years ago
|
||
http://lxr.mozilla.org/mozilla/source/other-licenses/branding/firefox/content/jar.mn And the unbranded equivalent is http://lxr.mozilla.org/mozilla/source/browser/base/jar.mn#47
Updated•19 years ago
|
Whiteboard: [eta 7/27]
Assignee | ||
Comment 6•19 years ago
|
||
Okay, this patch uses the branding content package.
Attachment #190483 -
Attachment is obsolete: true
Attachment #190658 -
Flags: review?(benjamin)
Updated•19 years ago
|
Attachment #190658 -
Flags: review?(benjamin)
Attachment #190658 -
Flags: review+
Attachment #190658 -
Flags: approval1.8b4+
Assignee | ||
Comment 7•19 years ago
|
||
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•