The default bug view has changed. See this FAQ.

build breaks when building SUTAgent with sdk level 11

RESOLVED FIXED in mozilla8

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: blassey, Assigned: blassey)

Tracking

Trunk
mozilla8
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

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

6 years ago
Created attachment 540621 [details] [diff] [review]
Patch to fix calls to setForeground in android

This should resolve the undefined symbol issue.
Attachment #540621 - Flags: review?(blassey.bugs)
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

6 years ago
Yes, I believe that it does go to the android log.
Assignee: nobody → blassey.bugs
Blocks: 676275
http://hg.mozilla.org/integration/mozilla-inbound/rev/a0f38e290e33
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/a0f38e290e33
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8

Comment 6

6 years ago
Created attachment 552118 [details] [diff] [review]
Patch to fix calls to setForeground in android in the watcher service code

This patch will take care of the setForeground api call in the watcher module for SUTAgent.
Attachment #552118 - Flags: review?(blassey.bugs)
(Assignee)

Updated

6 years ago
Attachment #552118 - Flags: review?(blassey.bugs) → review+
follow up patch on inbound
http://hg.mozilla.org/integration/mozilla-inbound/rev/d618c3a26cd4
(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
You need to log in before you can comment on or make changes to this bug.