1382774 - Fetch Android Gradle dependencies from jcenter rather than Maven Central

Status: RESOLVED FIXED

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

Description

[Nick Alexander :nalexander [he/him]]

Google is moving away from Maven Central and towards maven.google.com and jcenter.  We need to follow to pull recent dependencies in the Android Gradle dependencies job.

Comment 1

[Nick Alexander :nalexander [he/him]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=69bc9de08be9f8b65553c04640cd04dfd07a7fd8

Comment 2

[Nick Alexander :nalexander [he/him]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c945acabcedd98439a0ca0e26251bab1a41de197

Comment 3

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1382774 - Pre: Don't fail |mach android gradle-dependencies|.

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

Comment 4

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1382774 - Configure android-gradle-dependencies to use jcenter instead of central.

The nexus.xml included in this patch is the result of starting Nexus
and manually adding the jcenter proxy repository using the Nexus web
administration interface (all in a Docker container).  I know of no
way to do this configuration incrementally without the web interface.
The diff between new and the default generated configuration is a
single new <repository>..</repository> element.

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

Comment 5

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8888472 [details]
Bug 1382774 - Pre: Don't fail |mach android gradle-dependencies|.

https://reviewboard.mozilla.org/r/159418/#review165126

Comment 6

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8888473 [details]
Bug 1382774 - Configure android-gradle-dependencies to use jcenter instead of central.

https://reviewboard.mozilla.org/r/159420/#review165128

Comment 7

[Sebastian Kaspari (:sebastian; :pocmo)]

LGTM. I'll try to land them to unblock bug 1380639. :)

Comment 8

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8888473 [details]
Bug 1382774 - Configure android-gradle-dependencies to use jcenter instead of central.

https://reviewboard.mozilla.org/r/159420/#review165130

::: taskcluster/docker/android-gradle-build/bin/after.sh:17
(Diff revision 1)
>  # Not yet.  See notes on tooltool below.
>  # cp -R /root/.android-sdk android-sdk-linux
>  # tar cJf android-sdk-linux.tar.xz android-sdk-linux
>  
> -cp -R /workspace/nexus/storage/central jcentral
> +cp -R /workspace/nexus/storage/jcenter jcentral
>  tar cJf jcentral.tar.xz jcentral

Because of the name "jcentral" I thought we'd pull dependencies from jcenter already. :)

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/636f4f483359
Pre: Don't fail |mach android gradle-dependencies|. r=sebastian
https://hg.mozilla.org/integration/autoland/rev/2b78ed90d463
Configure android-gradle-dependencies to use jcenter instead of central. r=sebastian

Comment 10

[Nick Alexander :nalexander [he/him]]

(In reply to Sebastian Kaspari (:sebastian) from comment #8)
> Comment on attachment 8888473 [details]
> Bug 1382774 - Configure android-gradle-dependencies to use jcenter instead
> of central.
> 
> https://reviewboard.mozilla.org/r/159420/#review165130
> 
> ::: taskcluster/docker/android-gradle-build/bin/after.sh:17
> (Diff revision 1)
> >  # Not yet.  See notes on tooltool below.
> >  # cp -R /root/.android-sdk android-sdk-linux
> >  # tar cJf android-sdk-linux.tar.xz android-sdk-linux
> >  
> > -cp -R /workspace/nexus/storage/central jcentral
> > +cp -R /workspace/nexus/storage/jcenter jcentral
> >  tar cJf jcentral.tar.xz jcentral
> 
> Because of the name "jcentral" I thought we'd pull dependencies from jcenter
> already. :)

Yeah, me too.  I must have just been confused, since jcentral isn't jcenter and isn't Maven Central :)

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/636f4f483359
https://hg.mozilla.org/mozilla-central/rev/2b78ed90d463