Closed
Bug 1317089
Opened 6 years ago
Closed 5 years ago
[geckoview] Use Android's manifest-merger.jar to merge manifest fragments
Categories
(GeckoView :: General, defect)
GeckoView
General
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: nalexander, Unassigned)
References
Details
Attachments
(2 files)
Right now, we have a cobbled together system that: * uses the preprocessor to produce AndroidManifest.xml; * uses the preprocessor to product AppConstants.java; * uses build flags to include or exclude source files. Gradle (really, the Android build toolchain) has defined how to do this "for real". One declares build types, product flavors, and flavor dimensions, and uses an overlay system to produce a final AndroidManifest.xml, defines build config variables in Gradle files, and combines sets of Java, resource, and asset sources. It's nice! Gradle does some of this work internally (build config variables, source sets) but exposes a standalone manifest merger. This ticket tracks using that merger -- technically, the Android Manifest Merger (version 2) -- to assemble AndroidManifest.xml. It sucks to do this work rather than "just" use Gradle, but I think we'll need to keep both systems functional while we continue to try to switch over. Wish we could have made Aurora "just use" Gradle and figure out what's broken there.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8818783 [details] Bug 1317089 - Pre: Pass --debug-mode to aapt. https://reviewboard.mozilla.org/r/98736/#review99024 ::: mobile/android/base/Makefile.in:448 (Diff revision 1) > # We touch the target file before invoking aapt so that aapt's outputs > # are fresher than the target, preventing a subsequent invocation from > # thinking aapt's outputs are stale. This is safe because Make > # removes the target file if any recipe command fails. > > +# The --debug-mode flag below is intended to be: gps: any suggestion on how to express this cleanly appreciated.
Comment 4•6 years ago
|
||
mozreview-review |
Comment on attachment 8818783 [details] Bug 1317089 - Pre: Pass --debug-mode to aapt. https://reviewboard.mozilla.org/r/98736/#review99380
Attachment #8818783 -
Flags: review?(s.kaspari) → review+
Comment 5•6 years ago
|
||
mozreview-review |
Comment on attachment 8818784 [details] Bug 1317089 - Pre: Make MOZ_ANDROID_SHARED_ID mandatory. https://reviewboard.mozilla.org/r/98738/#review99382 It's so sad that we can't get rid of the shared user id anymore.
Attachment #8818784 -
Flags: review?(s.kaspari) → review+
Comment 6•6 years ago
|
||
mozreview-review-reply |
Comment on attachment 8818783 [details] Bug 1317089 - Pre: Pass --debug-mode to aapt. https://reviewboard.mozilla.org/r/98736/#review99024 > gps: any suggestion on how to express this cleanly appreciated. GNU make doesn't have boolean logic support. The closest you can do is eval something via a shell. But that's just as ugly. What you've done here is acceptable.
Comment 7•6 years ago
|
||
mozreview-review |
Comment on attachment 8818783 [details] Bug 1317089 - Pre: Pass --debug-mode to aapt. https://reviewboard.mozilla.org/r/98736/#review100746
Attachment #8818783 -
Flags: review?(gps) → review+
We use gradle now so this Just Works (mostly)
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
Updated•4 years ago
|
Product: Firefox for Android → GeckoView
You need to log in
before you can comment on or make changes to this bug.
Description
•