Closed Bug 789529 Opened 7 years ago Closed 7 years ago

Updater tests can't find channel-prefs.js

Categories

(Firefox :: General, defect)

x86_64
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 20

People

(Reporter: jimm, Assigned: jimm)

References

Details

(Whiteboard: completed-elm)

Attachments

(2 files, 1 obsolete file)

channel-prefs.js ends up in:

dist/bin/browser/defaults/pref/channel-prefs.js

which isn't found by update tests.

Moving it to 'preferences' solves the problem. But based on the comments in bug 756325 maybe that's not the right fix.

cc'ing a few people.
The code in the app makefile was updated to:

ifndef LIBXUL_SDK
# channel-prefs.js is handled separate from other prefs due to bug 756325
libs:: $(srcdir)/profile/channel-prefs.js
	$(NSINSTALL) -D $(FINAL_TARGET)/defaults/pref
	$(PYTHON) $(topsrcdir)/config/Preprocessor.py $(PREF_PPFLAGS) $(ACDEFINES) $^ > $(FINAL_TARGET)/defaults/pref/channel-prefs.js
endif


FINAL_TARGET points to the browser sub dir.
Moving this file back down to dist/bin also solves the problem. It also lets both browser share the same channel-pref file, so maybe that's the right fix?
This is filed as Win7 but isn't this a Win8 bug?

The channel-prefs.js needs to end up in the defaults/pref directory that is read by the app per bug 756325.

Nick, I'm still tempted to get rid of channel-prefs.js entirely and go with allowing QA to provide an override other than the channel-prefs.js file. If you ever need to change the channel that would still be possible when updating.
(In reply to Jim Mathies [:jimm] from comment #2)
> Moving this file back down to dist/bin also solves the problem. It also lets
> both browser share the same channel-pref file, so maybe that's the right fix?
I think that is the better way to go for now. If nothing else, they are the same install and there shouldn't be the possibility of having two different channels based due to two channel-prefs.js files.
(In reply to Robert Strong [:rstrong] (do not email) from comment #3)
> This is filed as Win7 but isn't this a Win8 bug?

This is fall out from the metro-build bug, so once that land on mc this will show up there.

> I think that is the better way to go for now. If nothing else, they are the
> same install and there shouldn't be the possibility of having two different
> channels based due to two channel-prefs.js files.

Ok, I'll land that change on elm to be sure it addresses the update test failures. Will post a patch here as well for Glandium.
Attached patch patchSplinter Review
This just reverts a single change in the "Move browser application in a subdirectory (PoC)" patch in bug 755724.
Attached patch packager (obsolete) — Splinter Review
revert packager changes as well.
Attached patch packagerSplinter Review
Attachment #659341 - Attachment is obsolete: true
Whiteboard: completed-elm
No longer blocks: metro-build
Comment on attachment 659304 [details] [diff] [review]
patch

This is needed on m-c, as bug 787180 landed there a while ago. This is effectively a noop until FINAL_TARGET != DIST/bin.
Attachment #659304 - Flags: review+
Assignee: nobody → jmathies
https://hg.mozilla.org/mozilla-central/rev/4fe60dd3d145
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Blocks: 825872
No longer blocks: 825872
You need to log in before you can comment on or make changes to this bug.