Closed
Bug 737167
Opened 12 years ago
Closed 12 years ago
Build changes to support gonk-ics
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mwu, Assigned: mwu)
References
Details
Attachments
(2 files)
3.34 KB,
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
1.25 KB,
patch
|
cjones
:
review-
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
I seem to get build issues in JS without switching to a newer toolchain. This patch lets us easily switch between toolchains.
Attachment #607288 -
Flags: review?
Assignee | ||
Comment 2•12 years ago
|
||
This patch sets MOZ_B2G_ICS when we're building against an ICS tree.
Attachment #607289 -
Flags: review?(jones.chris.g)
Attachment #607289 -
Flags: review?(gal)
Assignee | ||
Updated•12 years ago
|
Attachment #607288 -
Flags: review?(jones.chris.g)
Attachment #607288 -
Flags: review?(gal)
Attachment #607288 -
Flags: review?
Comment 3•12 years ago
|
||
We decided to do runtime detection instead so we can auto update one gecko.
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Andreas Gal :gal from comment #3) > We decided to do runtime detection instead so we can auto update one gecko. Runtime detection doesn't help when the headers move. Most of these ifdefs are used just to make things compile, but if done right, shouldn't affect our ability to ship a single binary on both GB and ICS.
Updated•12 years ago
|
Attachment #607288 -
Flags: review?(jones.chris.g) → review+
Comment on attachment 607289 [details] [diff] [review] 2/2: Autodetect ICS I don't want a compile-time GB/ICS switch unless it's absolutely necessary. Convince me that this is absolutely necessary.
Attachment #607289 -
Flags: review?(jones.chris.g) → review-
Comment 6•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/33fa8d55914c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•12 years ago
|
Attachment #607288 -
Flags: review?(gal)
Assignee | ||
Updated•12 years ago
|
Attachment #607289 -
Flags: review?(gal)
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•