Closed Bug 1509572 Opened 5 years ago Closed 5 years ago

Cleanup small pieces of Gradle integration

Categories

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

enhancement
Not set
normal

Tracking

(firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: nalexander, Assigned: nalexander)

References

(Blocks 1 open bug)

Details

Attachments

(6 files)

This is just a ticket to land some Pre: parts for Bug 1509539.
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
This depends on there being only one Gradle version downloaded, but I
think that is safe enough.
--console=plain is critical in automation, but it's not always helpful
locally.  Let's try without it for a while.

--continue tries to build additional tasks even after the first
failing task.  It can be useful in automation ('cuz you might see more
errors) but it can also be confusing (because the build fails but
produces additional output).  Let's try without it, too.

Depends on D12790
This has never been as useful as anticipated: we really aren't seeing
resource mismatches in the wild that need diagnostic aids.

Depends on D12791
I'm not sure how this ever worked in the `android-gradle-dependencies`
task -- it must have been because of `--continue`.

Depends on D12792
This will be exploited later, when we start making the
`withGeckoBinaries` switch conditional on the tasks that Gradle is
going to execute.

Depends on D12793
Before, we always ran the "generate JNI wrappers" command and had the
command be smart about updating the output.  Now we move the smarts to
the Gradle side to streamline the build.

Depends on D12794
Pushed by nalexander@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c4087e8ea340
Part 1: Bump Gradle wrapper to 4.10.2; decouple script from Gradle version. r=snorp
https://hg.mozilla.org/integration/autoland/rev/bd0729ff0bbc
Part 2: Don't pass --console=plain and --continue through to Gradle. r=snorp
https://hg.mozilla.org/integration/autoland/rev/bc6911d14ae0
Part 3: Don't dump R.txt for every build. r=snorp
https://hg.mozilla.org/integration/autoland/rev/122d6f1a1136
Part 4: Make `geckoview:generateSDKBindings` work without arguments. r=snorp
https://hg.mozilla.org/integration/autoland/rev/f6653c7d2967
Part 5: Depend on bundle, not assemble. r=snorp
https://hg.mozilla.org/integration/autoland/rev/9f6a7e664cef
Part 6: Don't always generate annotations from Gradle. r=snorp
Pushed by nalexander@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c145e1dbd4d
Part 1: Bump Gradle wrapper to 4.10.2; decouple script from Gradle version. r=snorp
https://hg.mozilla.org/integration/autoland/rev/f84f04e601e4
Part 2: Don't pass --console=plain and --continue through to Gradle. r=snorp
https://hg.mozilla.org/integration/autoland/rev/a748fb34ae07
Part 3: Don't dump R.txt for every build. r=snorp
https://hg.mozilla.org/integration/autoland/rev/851fd4b655df
Part 4: Make `geckoview:generateSDKBindings` work without arguments. r=snorp
https://hg.mozilla.org/integration/autoland/rev/d8102b5a209a
Part 5: Depend on bundle, not assemble. r=snorp
https://hg.mozilla.org/integration/autoland/rev/8d0de38dc4c8
Part 6: Don't always generate annotations from Gradle. r=snorp
(In reply to Dorel Luca [:dluca] from comment #8)
> Backed out 6 changesets (bug 1509572) for linting failure. CLOSED TREE
> 
> Log:
> https://treeherder.mozilla.org/logviewer.
> html#?job_id=213957496&repo=autoland&lineNumber=272
> 
> Push with failures:
> https://treeherder.mozilla.org/#/
> jobs?repo=autoland&group_state=expanded&revision=9f6a7e664cef95703e629aa3fd02
> cd6a5b014f54
> 
> Backout:
> https://hg.mozilla.org/integration/autoland/rev/
> 79ae8ec996e64730196a994093f3ff8bb4af1384

Thanks, Dorel.  I folded a fix into a future commit; back-ported and relanded.  Fingers crossed!
Flags: needinfo?(nalexander)
Regressions: 1540820
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 65 → mozilla65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: