Closed
Bug 665716
Opened 13 years ago
Closed 13 years ago
build breaks when building SUTAgent with sdk level 11
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla8
People
(Reporter: blassey, Assigned: blassey)
References
Details
Attachments
(2 files)
|
(deleted),
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
|
(deleted),
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
if your mozconfig specifies:
ac_add_options --with-android-sdk="<path to your sdk>/platforms/android-11"
the build will break when building the SUTAgent with:
symbol : method setForeground(boolean)
location: class com.mozilla.SUTAgentAndroid.service.ASMozStub
setForeground(true);
^
/home/vmuser/Android/MC-18-06-2011/build/mobile/sutagent/android/ASMozStub.java:213: cannot find symbol
symbol : method setForeground(boolean)
location: class com.mozilla.SUTAgentAndroid.service.ASMozStub
setForeground(false);
^
2 errors
make[6]: *** [classes.dex] Error 1
|
||
Comment 1•13 years ago
|
||
This should resolve the undefined symbol issue.
Attachment #540621 -
Flags: review?(blassey.bugs)
|
Assignee | |
Comment 2•13 years ago
|
||
Comment on attachment 540621 [details] [diff] [review]
Patch to fix calls to setForeground in android
We use Log.e() to log errors in regular fennec code. Does Exception.printStackTrace() go the the android log? Or does it just get lost in stderr?
Attachment #540621 -
Flags: review?(blassey.bugs) → review+
|
|
||
Comment 3•13 years ago
|
||
Yes, I believe that it does go to the android log.
|
|
Assignee | |
Comment 4•13 years ago
|
||
Whiteboard: [inbound]
|
||
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
|
|
||
Comment 6•13 years ago
|
||
This patch will take care of the setForeground api call in the watcher module for SUTAgent.
Attachment #552118 -
Flags: review?(blassey.bugs)
|
|
Assignee | |
Updated•13 years ago
|
Attachment #552118 -
Flags: review?(blassey.bugs) → review+
|
|
Assignee | |
Comment 7•13 years ago
|
||
follow up patch on inbound
http://hg.mozilla.org/integration/mozilla-inbound/rev/d618c3a26cd4
|
||
Comment 8•13 years ago
|
||
(In reply to Brad Lassey [:blassey][blassey@mozilla.com] from comment #7)
> follow up patch on inbound
> http://hg.mozilla.org/integration/mozilla-inbound/rev/d618c3a26cd4
and merged:
http://hg.mozilla.org/mozilla-central/rev/d618c3a26cd4
|
||
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•