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)

enhancement
Not set
normal

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.
The build system no longer invokes these directly: they're all fetched and invoked by Gradle and its plugins.
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
Nothing is actually using the value of EMULATOR from the build system. Depends on D14288
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
Attachment #9030839 - Attachment description: Bug 1498406 - Part 4: Purge unused ANDROID_BUILD_TOOLS_VERSION. → Bug 1498406 - Part 3: Purge unused ANDROID_BUILD_TOOLS_VERSION.
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
Assignee: nobody → nalexander
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.

Attachment

General

Created:
Updated:
Size: