360941 - UpdateDriver and UpdateService update location mismatch

Status: RESOLVED FIXED

Description

[Wolfgang Rosenauer [:wolfiR]]

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)

Comment 1

[Wolfgang Rosenauer [:wolfiR]]

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.

Comment 2

[Wolfgang Rosenauer [:wolfiR]]

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

This patch should at least bring back the functionality like in 1.8.1

Comment 3

[Wolfgang Rosenauer [:wolfiR]]

/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.

Comment 4

[Benjamin Smedberg]

This needs a unit test as well. Sorry for not catching that earlier.