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)
Firefox Build System
General
Tracking
(firefox14 fixed, firefox15 fixed)
RESOLVED
FIXED
mozilla16
People
(Reporter: sgautherie, Assigned: glandium)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.53 KB,
patch
|
khuey
:
review+
sgautherie
:
feedback+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
(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.
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #639223 -
Flags: review?(khuey)
Reporter | ||
Comment 2•13 years ago
|
||
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+
Attachment #639223 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Target Milestone: --- → Firefox 16
Assignee | ||
Comment 4•13 years ago
|
||
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 5•13 years ago
|
||
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+
Comment 6•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•13 years ago
|
Flags: in-testsuite-
Assignee | ||
Comment 7•13 years ago
|
||
Updated•7 years ago
|
Component: Build Config → General
Product: Firefox → Firefox Build System
Updated•7 years ago
|
Keywords: regression
Target Milestone: Firefox 16 → mozilla16
Updated•7 years ago
|
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•