Closed
Bug 1213170
Opened 9 years ago
Closed 9 years ago
Intermittent gmake[4]: *** [mobile/android/base/libs] Error 2 on Taskcluster builds
Categories
(Taskcluster :: General, defect)
Taskcluster
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nigelb, Unassigned)
References
()
Details
Attachments
(1 file)
16.12 KB,
patch
|
reuben
:
review+
|
Details | Diff | Splinter Review |
https://treeherder.mozilla.org/logviewer.html#?job_id=15434307&repo=mozilla-inbound 03:44:14 INFO - /home/worker/workspace/build/src/android-sdk-linux/build-tools/23.0.1/dx --dex --output=classes.dex jars-proguarded 03:44:26 INFO - ../../../config/nsinstall -R classes.dex ../../../dist/bin 03:44:26 INFO - ***************************************************** 03:44:26 INFO - *** Error: The generated JNI code has changed *** 03:44:26 INFO - * To update generated code in the tree, please run * 03:44:26 INFO - make -C /home/worker/workspace/build/src/obj-firefox/mobile/android/base update-generated-wrappers 03:44:26 INFO - * Repeat the build, and check in any changes. * 03:44:26 INFO - ***************************************************** 03:44:26 INFO - gmake[5]: *** [libs] Error 1 03:44:26 INFO - gmake[5]: Leaving directory `/home/worker/workspace/build/src/obj-firefox/mobile/android/base' 03:44:26 INFO - gmake[4]: *** [mobile/android/base/libs] Error 2 03:44:26 INFO - gmake[4]: Leaving directory `/home/worker/workspace/build/src/obj-firefox' 03:44:26 INFO - gmake[3]: *** [libs] Error 2 03:44:26 INFO - gmake[3]: Leaving directory `/home/worker/workspace/build/src/obj-firefox' 03:44:26 INFO - gmake[2]: *** [default] Error 2 03:44:26 INFO - gmake[2]: Leaving directory `/home/worker/workspace/build/src/obj-firefox' 03:44:26 INFO - gmake[1]: *** [realbuild] Error 2 03:44:26 INFO - gmake[1]: Leaving directory `/home/worker/workspace/build/src' 03:44:26 INFO - gmake: *** [build] Error 2 03:44:26 INFO - 393 compiler warnings present. 03:44:27 INFO - Notification center failed: Install the python dbus module to get a notification when the build finishes. 03:44:27 ERROR - Return code: 2 03:44:27 WARNING - setting return code to 2 03:44:27 FATAL - 'mach build' did not run successfully. Please check log for errors. 03:44:27 FATAL - Running post_fatal callback... 03:44:27 FATAL - Exiting -1 03:44:27 INFO - Running post-action listener: influxdb_recording_post_action 03:44:27 INFO - Running post-action listener: record_mach_stats 03:44:27 INFO - Running post-run listener: _summarize 03:44:27 ERROR - # TBPL FAILURE # 03:44:27 INFO - ##### 03:44:27 INFO - ##### FxDesktopBuild summary: 03:44:27 INFO - ##### 03:44:27 ERROR - # TBPL FAILURE #
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Reporter | ||
Comment 3•9 years ago
|
||
jlund, I'm told you would be able to help since it's b2gdroid.
Flags: needinfo?(jlund)
Comment hidden (Intermittent Failures Robot) |
Comment 5•9 years ago
|
||
Fairly sure this is fallout from bug 1197010, where I flipped MOZ_WEBSMS_BACKEND on B2GDroid. See https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/Makefile.in#216 Looks like the long term solution is bug 839587.
Comment 6•9 years ago
|
||
Attachment #8672919 -
Flags: review?(nalexander)
Comment 8•9 years ago
|
||
Comment on attachment 8672919 [details] [diff] [review] Always auto-genereate stubs for GeckoSmsManager Review of attachment 8672919 [details] [diff] [review]: ----------------------------------------------------------------- We don't want this to be included in Fennec, right? And it's not clear to me that this will work for b2gdroid and Fennec as written. Please push a try build. Fundamentally the JNI work for Android never envisioned multiple products. We'll have to do some thinking about how to support this better; perhaps ckitching's old "real annotation processor" approach in Bug 1067217 is needed.
Comment 9•9 years ago
|
||
Comment on attachment 8672919 [details] [diff] [review] Always auto-genereate stubs for GeckoSmsManager Review of attachment 8672919 [details] [diff] [review]: ----------------------------------------------------------------- Sorry, r- until I see the try and understand more; r+ after discussion, I expect.
Attachment #8672919 -
Flags: review?(nalexander) → review-
Comment 10•9 years ago
|
||
This is jchen territory. jchen, do you have a vision for how this should be addressed? If not, I can add this to my list.
Flags: needinfo?(nchen)
Comment 11•9 years ago
|
||
I agree with Bug 1067217 being the proper solution, but I don't know how much work that bug requires. In the meantime, the patch here will increase the size of libmozglue.so for Fennec slightly, but I think it's okay as a trivial workaround. We could figure out how to generate two sets of bindings, but I think that time is probably better spent working on something else, perhaps Bug 1067217.
Flags: needinfo?(nchen)
Comment 15•9 years ago
|
||
Comment on attachment 8672919 [details] [diff] [review] Always auto-genereate stubs for GeckoSmsManager 14:52 <nalexander> reuben: fabrice: jchen's cool with that work-around, bombs away. Looks like you already landed. Flip the review flag for me, pleas3e.
Attachment #8672919 -
Flags: review- → review+
Comment 16•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/4049dd2bf126 https://hg.mozilla.org/integration/b2g-inbound/rev/ac9f96ca40cb
Comment hidden (Intermittent Failures Robot) |
Comment 18•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8b0dcdfa6341 https://hg.mozilla.org/mozilla-central/rev/2bd66074e01a https://hg.mozilla.org/mozilla-central/rev/4049dd2bf126 https://hg.mozilla.org/mozilla-central/rev/ac9f96ca40cb
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 21•9 years ago
|
||
For everybody getting this bugspam, the graph in brasstacks suggests that this isn't intermittent any longer -- it's really fixed.
Updated•9 years ago
|
Keywords: intermittent-failure
You need to log in
before you can comment on or make changes to this bug.
Description
•