1252928 - Use in-tree Fennec Gradle configuration rather than special dependencies configuration in Android Gradle dependency fetching task

Status: RESOLVED FIXED

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

Description

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

I landed https://dxr.mozilla.org/mozilla-central/source/testing/docker/android-gradle-build/project to quickly get dependency fetching into the tree.  We'd much rather use the actual project in https://dxr.mozilla.org/mozilla-central/source/mobile/android/app/build.gradle and add sdk-manager-plugin there.  This ticket tracks doing that work.

Comment 1

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=51198adaacdb shows this working.

Comment 2

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

Attachment: MozReview Request: Bug 1252928 - Part 1: Use Jake Wharton's sdk-manager-plugin in Fennec's Gradle configuration. r?sebastian

This is a version produced locally from
https://github.com/JakeWharton/sdk-manager-plugin/commit/1d07987217120fffdc0ce1291d73e974310d7d92.
I used |./gradlew install| and then copied the pieces out of ~/.m2/repository.

Local builders should never download using this plugin, since |mach configure|
ensures that all the relevant pieces can already be found.  It's just useful in
automation; and as an eventual replacement for chunks of |mach bootstrap|.

Review commit: https://reviewboard.mozilla.org/r/38813/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38813/

Comment 3

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

Attachment: MozReview Request: Bug 1252928 - Part 2: Turn android-gradle-build into a desktop-build clone. r?dustin

This container formerly defined a special Gradle project for fetching
dependencies.  This patch lays the ground-work to use the in-tree
Gradle project instead.  Using the in-tree project looks like first
starting a local Nexus repository to collect downloaded dependencies
(before.sh); then running a build which populates that repository; and
then packaging up the downloaded dependencies (after.sh).  The patch
after this will define the build which populates the repository.

Sadly there's no easy way to *inherit* from desktop-build, so this is
a copy-paste-modify clone.

Review commit: https://reviewboard.mozilla.org/r/38815/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38815/

Comment 4

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

Attachment: MozReview Request: Bug 1252928 - Part 3: Replace android-gradle-build-dependencies with android-api-15-gradle-dependencies. r?dustin

The existing task's outcome is best achieved with a special purpose
build task, so here it is.

Review commit: https://reviewboard.mozilla.org/r/38817/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38817/

Comment 5

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8728145 [details]
MozReview Request: Bug 1252928 - Part 1: Use Jake Wharton's sdk-manager-plugin in Fennec's Gradle configuration. r?sebastian

https://reviewboard.mozilla.org/r/38813/#review35573

Comment 6

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 8728146 [details]
MozReview Request: Bug 1252928 - Part 2: Turn android-gradle-build into a desktop-build clone. r?dustin

https://reviewboard.mozilla.org/r/38815/#review35597

Comment 7

[Dustin J. Mitchell [:dustin] (he/him)]

Comment on attachment 8728147 [details]
MozReview Request: Bug 1252928 - Part 3: Replace android-gradle-build-dependencies with android-api-15-gradle-dependencies. r?dustin

https://reviewboard.mozilla.org/r/38817/#review35595

::: testing/taskcluster/scripts/misc/repackage-jdk-centos.sh:27
(Diff revision 1)
> +ln -s /etc/pki/java/cacerts java_home/jre/lib/security/cacerts

It occurs to me, tar can do this too:
  http://www.gnu.org/software/tar/manual/html_node/dereference.html
(assuming the link is correc to begin with)

::: testing/taskcluster/tasks/builds/android_api_15_gradle_dependencies.yml:60
(Diff revision 1)
> +        /home/worker/bin/after.sh &&

I like this!

That's minor - if you end up doing another round of review, maybe switch to use `tar --dereference`, but otherwise this is fine.

Comment 8

[Nick Alexander :nalexander [he/him] (PTO Feb 24-March 3)]

(In reply to Dustin J. Mitchell [:dustin] from comment #7)
> Comment on attachment 8728147 [details]
> MozReview Request: Bug 1252928 - Part 3: Replace
> android-gradle-build-dependencies with android-api-15-gradle-dependencies.
> r?dustin
> 
> https://reviewboard.mozilla.org/r/38817/#review35595
> 
> ::: testing/taskcluster/scripts/misc/repackage-jdk-centos.sh:27
> (Diff revision 1)
> > +ln -s /etc/pki/java/cacerts java_home/jre/lib/security/cacerts
> 
> It occurs to me, tar can do this too:
>   http://www.gnu.org/software/tar/manual/html_node/dereference.html
> (assuming the link is correc to begin with)
> 
> :::
> testing/taskcluster/tasks/builds/android_api_15_gradle_dependencies.yml:60
> (Diff revision 1)
> > +        /home/worker/bin/after.sh &&
> 
> I like this!
> 
> That's minor - if you end up doing another round of review, maybe switch to
> use `tar --dereference`, but otherwise this is fine.

Just for the next person: there's a subtle difference between making the symlink absolute and using --dereference.  Making the symlink absolute lets the consuming system set the cacerts; using --dereference takes the producing system's cacerts and sets them in stone.  Until further notice, I prefer the former.

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/d316d6160a5f
https://hg.mozilla.org/integration/fx-team/rev/022cdd61e207
https://hg.mozilla.org/integration/fx-team/rev/063fb6e87b4e

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d316d6160a5f
https://hg.mozilla.org/mozilla-central/rev/022cdd61e207
https://hg.mozilla.org/mozilla-central/rev/063fb6e87b4e