UpdateDriver and UpdateService update location mismatch

RESOLVED FIXED

Status

RESOLVED FIXED
12 years ago
3 years ago

People

(Reporter: wolfiR, Unassigned)

Tracking

Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
hmm, bad summary.

Explanation:
This only affects trunk and not 1.8 and I already discussed it with Ben Turner via PM.
Current trunk updater doesn't work for xulrunner applications which ship its own xulrunner because the nsUpdateService components downloads updates to XCurProcD which is the application directory on trunk (likely changed by bug 319843) but nsUpdateDriver is searching updates in appDir/xulrunner/updates/ because the GREDir is appDir/xulrunner.

Trying to fix this would need some design questions answered.
I think that appDir/updates is the right location and therefore nsUpdateDriver could be changed to scan this directory if it runs a xulapp.
(That's what I use at the moment as workaround/bugfix)

In addition to that the update driver should copy in a copy of updater.ini which resides in the appDir and not the GREDir (appDir/xulrunner)
(Reporter)

Comment 1

12 years ago
Created attachment 246368 [details] [diff] [review]
patch #1

This one more or less restores the functionality in 1.8.1 while updates are now downloaded to appDir/updates and run from there.
(Reporter)

Comment 2

12 years ago
Comment on attachment 246368 [details] [diff] [review]
patch #1

This patch should at least bring back the functionality like in 1.8.1
Attachment #246368 - Flags: first-review?(benjamin)

Updated

12 years ago
Attachment #246368 - Flags: first-review?(benjamin) → first-review+
(Reporter)

Comment 3

12 years ago
/cvsroot/mozilla/toolkit/xre/nsUpdateDriver.cpp,v  <--  nsUpdateDriver.cpp
new revision: 1.21; previous revision: 1.20

The mismatch is fixed and an updater.ini move could be discussed in another bug.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 4

12 years ago
This needs a unit test as well. Sorry for not catching that earlier.
Flags: in-testsuite?
(Assignee)

Updated

3 years ago
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.