Closed Bug 1260914 Opened 4 years ago Closed 4 years ago

Correct android-lint build config

Categories

(Firefox Build System :: Android Studio and Gradle Integration, defect)

All
Android
defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: mcomella, Assigned: mcomella)

References

Details

Attachments

(1 file)

Apparently it's using the 64_api_15_frontend sub config so it's actually performing unit test jobs.

Context:
13:52 <mcomella> nalexander: Hm, I just noticed lint is using the wrong subconfig – do you know how that's inferred?
13:52 <mcomella> Context: my switch to 'android-test' failed with a warning that tried to use the frontend sub config: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3ff124ee8550&selectedJob=18823777
13:52 <mcomella> So I looked at my original lint/test test push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3370dfbb8892&selectedJob=18765323
13:53 <mcomella> And when you click into the artifacts: https://tools.taskcluster.net/task-inspector/#GBKMJwliQ4WihDnaQjOU2g/0
13:53 <mcomella> It says the file is missing on the worker

13:58 <mcomella> nalexander: aha, may have found it: https://mxr.mozilla.org/mozilla-central/source/testing/mozharness/mozharness/mozilla/building/buildbase.py#361
Assignee: nobody → michael.l.comella
Comment on attachment 8736511 [details]
MozReview Request: Bug 1260914 - Correct android-lint TC task. r=nalexander

https://reviewboard.mozilla.org/r/43329/#review39945

r+ via irc.
Attachment #8736511 - Flags: review+
Comment on attachment 8736511 [details]
MozReview Request: Bug 1260914 - Correct android-lint TC task. r=nalexander

https://reviewboard.mozilla.org/r/43329/#review39947
Attachment #8736511 - Flags: review?(nalexander) → review+
https://hg.mozilla.org/mozilla-central/rev/320b39130851
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 48 → mozilla48
You need to log in before you can comment on or make changes to this bug.