Closed
Bug 1260914
Opened 9 years ago
Closed 9 years ago
Correct android-lint build config
Categories
(Firefox Build System :: Android Studio and Gradle Integration, defect)
Firefox Build System
Android Studio and Gradle Integration
All
Android
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 | ||
Comment 1•9 years ago
|
||
| Assignee | ||
Updated•9 years ago
|
Assignee: nobody → michael.l.comella
| Assignee | ||
Comment 2•9 years ago
|
||
The missing lint:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8c2d03a429f6
| Assignee | ||
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/43329/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/43329/
Attachment #8736511 -
Flags: review?(nalexander)
| Assignee | ||
Comment 4•9 years ago
|
||
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 5•9 years ago
|
||
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+
| Assignee | ||
Comment 6•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/320b39130851d20d0f4dd654e1b6f8f322c686fb
Bug 1260914 - Correct android-lint TC task. r=nalexander
Comment 7•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•6 years ago
|
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.
Description
•