Closed
Bug 794297
Opened 12 years ago
Closed 12 years ago
(Full) b2g builds broken by bug 791775
Categories
(Toolkit :: Crash Reporting, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: cjones, Assigned: cjones)
Details
Attachments
(1 file)
816 bytes,
patch
|
mwu
:
review+
|
Details | Diff | Splinter Review |
build/core/base_rules.mk:82: *** Module name: test_google_breakpad build/core/base_rules.mk:83: *** Makefile location: gecko/toolkit/crashreporter/google-breakpad/android/sample_app/jni build/core/base_rules.mk:84: * build/core/base_rules.mk:85: * Each module must use a LOCAL_MODULE_TAGS in its build/core/base_rules.mk:86: * Android.mk. Possible tags declared by a module: The breakpad update pulled in some Android.mk turds we don't want. Need to block them.
Assignee | ||
Comment 1•12 years ago
|
||
Assignee: nobody → jones.chris.g
Attachment #664707 -
Flags: review?(mwu)
Updated•12 years ago
|
Attachment #664707 -
Flags: review?(mwu) → review+
Assignee | ||
Comment 2•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/acd23e460291 Need to get this merged to inbound asap.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 3•12 years ago
|
||
catlee landed a separate fix for this in bug 777530, just hg rm'ing the Android.mk files in question.
Assignee | ||
Comment 4•12 years ago
|
||
That can make updating more annoying, and doesn't fix the next case when a subproject we import adds more Android.mk turds.
Comment 5•12 years ago
|
||
For next time we could either throw an empty Android.mk in the m-c root, or relocate gecko/ in the B2G tree under gonk-misc/ or somewhere else that already has a Android.mk that does not recurse. The former is surely less painful for B2G folks.
Comment 6•12 years ago
|
||
oh nm. that's exactly what you did :)
You need to log in
before you can comment on or make changes to this bug.
Description
•