Closed Bug 769130 Opened 13 years ago Closed 13 years ago

browser/app/Makefile.in: setting PREF_JS_EXPORTS _after_ including rules.mk is useless

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox14 fixed, firefox15 fixed)

RESOLVED FIXED
mozilla16
Tracking Status
firefox14 --- fixed
firefox15 --- fixed

People

(Reporter: sgautherie, Assigned: glandium)

References

Details

(Keywords: regression)

Attachments

(1 file)

(In reply to Mike Hommey [:glandium] from bug 762588 comment #9) > (In reply to Serge Gautherie (:sgautherie) from comment #8) > > Comment on attachment 631062 [details] [diff] [review] > > Install channel-prefs.js in PREF_DIR when building Firefox as a XULRunner app > > > > Review of attachment 631062 [details] [diff] [review]: > > ----------------------------------------------------------------- > > > > ::: browser/app/Makefile.in > > @@ +142,5 @@ > > > libs:: $(srcdir)/profile/channel-prefs.js > > > $(NSINSTALL) -D $(DIST)/bin/defaults/pref > > > $(PYTHON) $(topsrcdir)/config/Preprocessor.py $(PREF_PPFLAGS) $(ACDEFINES) $^ > $(DIST)/bin/defaults/pref/channel-prefs.js > > > +else > > > +PREF_JS_EXPORTS += $(srcdir)/profile/channel-prefs.js > > > > I'm a little confused. > > Isn't it useless to set PREF_JS_EXPORTS _after_ including rules.mk? > > http://mxr.mozilla.org/comm-central/search?string=PREF_JS_EXPORTS&case=on > > Mmmm you're right.
Attachment #639223 - Flags: review?(khuey)
Comment on attachment 639223 [details] [diff] [review] Actually install channel-prefs.js in PREF_DIR when building Firefox as a XULRunner app This looks like what I expected.
Attachment #639223 - Flags: feedback+
Comment on attachment 639223 [details] [diff] [review] Actually install channel-prefs.js in PREF_DIR when building Firefox as a XULRunner app [Approval Request Comment] Bug caused by bug 762588 and initially bug 756325. User impact if declined: channel-prefs.js is not shipped at all after bug 762588. (only affects Firefox as a xulrunner application, not mozilla builds, nor local builds) Risk to taking this patch (and alternatives if risky): NPOTB. String or UUID changes made by this patch: None.
Attachment #639223 - Flags: approval-mozilla-beta?
Attachment #639223 - Flags: approval-mozilla-aurora?
Comment on attachment 639223 [details] [diff] [review] Actually install channel-prefs.js in PREF_DIR when building Firefox as a XULRunner app [Triage Comment] NPOTB, approved for Aurora 15 and Beta 14.
Attachment #639223 - Flags: approval-mozilla-beta?
Attachment #639223 - Flags: approval-mozilla-beta+
Attachment #639223 - Flags: approval-mozilla-aurora?
Attachment #639223 - Flags: approval-mozilla-aurora+
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
Component: Build Config → General
Product: Firefox → Firefox Build System
Keywords: regression
Target Milestone: Firefox 16 → mozilla16
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: