Closed Bug 1440432 Opened 2 years ago Closed 2 years ago

Remove AAR exploding from configure

Categories

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

enhancement
Not set

Tracking

(firefox60 fixed)

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: nalexander, Assigned: nalexander)

References

Details

Attachments

(1 file)

Way back in https://bugzilla.mozilla.org/show_bug.cgi?id=1108782, we "exploded" AAR files at configure time in order to capture correct dependencies for the moz.build system to consume.  That approach worked really, really well -- but it's time to go, since Gradle works really, really, really (!) well.

There's a bunch of pieces here that just need to be deleted and simplified.  In particular, all of the *AAR_LIB variables can go.  I think we'll want to capture certain versions in /build.gradle rather than configure, too.

This should simplify our configure code, and also speed up every |mach configure| with --enable-application=mobile/android.
Depends on: 1440433
No longer depends on: 1440433
Comment on attachment 8953610 [details]
Bug 1440432 - Remove AAR exploding from configure.

https://reviewboard.mozilla.org/r/222816/#review229268
Attachment #8953610 - Flags: review?(nchen) → review+
Depends on: 1440757
Assignee: nobody → nalexander
Blocks: 1255924
Status: NEW → ASSIGNED
Pushed by nalexander@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b48de6629c9d
Remove AAR exploding from configure. r=jchen
https://hg.mozilla.org/mozilla-central/rev/b48de6629c9d
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 60 → mozilla60
You need to log in before you can comment on or make changes to this bug.