Closed
Bug 1498406
Opened 6 years ago
Closed 6 years ago
Streamline Android configure post Gradle
Categories
(Firefox Build System :: Android Studio and Gradle Integration, enhancement)
Firefox Build System
Android Studio and Gradle Integration
Tracking
(firefox66 fixed)
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: nalexander, Assigned: nalexander)
References
(Blocks 1 open bug)
Details
Attachments
(3 files, 1 obsolete file)
The Android build landscape has totally changed now that we always use Gradle to build. We no longer invoke:
- javac
- javah
- jar
at all: all compilation and packaging is handled by Gradle and its plugins, and those plugins are fetched dynamically. We no longer require a particular Android platform, nor a particular version of the Android build-tools, for the same reasons -- they are all fetched dynamically.
This should all mean that we can simplify https://searchfox.org/mozilla-central/source/build/autoconf/android.m4 radically.
Assignee | ||
Comment 1•6 years ago
|
||
The build system no longer invokes these directly: they're all fetched
and invoked by Gradle and its plugins.
Assignee | ||
Comment 2•6 years ago
|
||
There's no need -- it will be fetched by Gradle when needed. The only
wrinkle is that the user might be asked to confirm license agreements,
which could be a hassle (and could cause problems in CI, but there we
install relevant platforms in advance).
Depends on D14287
Assignee | ||
Comment 3•6 years ago
|
||
Nothing is actually using the value of EMULATOR from the build system.
Depends on D14288
Assignee | ||
Comment 4•6 years ago
|
||
We don't care anymore: the Gradle build fetches an appropriate version
of the build-tools package, and the one build-tool we actually use --
zipalign -- isn't version sensitive.
Depends on D14289
Updated•6 years ago
|
Attachment #9030839 -
Attachment description: Bug 1498406 - Part 4: Purge unused ANDROID_BUILD_TOOLS_VERSION. → Bug 1498406 - Part 3: Purge unused ANDROID_BUILD_TOOLS_VERSION.
Updated•6 years ago
|
Attachment #9030838 -
Attachment is obsolete: true
Pushed by nalexander@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/df01499f9fdf
Part 1: Don't check for javac, javah, jar. r=froydnj
https://hg.mozilla.org/integration/autoland/rev/cc64661d5c94
Part 2: Don't check for Android platform in configure. r=froydnj
https://hg.mozilla.org/integration/autoland/rev/b0002d2ea03f
Part 3: Purge unused ANDROID_BUILD_TOOLS_VERSION. r=froydnj
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/df01499f9fdf
https://hg.mozilla.org/mozilla-central/rev/cc64661d5c94
https://hg.mozilla.org/mozilla-central/rev/b0002d2ea03f
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Updated•6 years ago
|
Assignee: nobody → nalexander
Updated•5 years ago
|
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 66 → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•