Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use gradle when the build target is Aab #156

Draft
wants to merge 1 commit into
base: gradle-library-dependencies
Choose a base branch
from

Conversation

MarijnS95
Copy link
Member

Depends on #140 to prevent conflicts

Instead of explicitly requiring gradle: true in the manifest, enable it by default when the (implicit or explicit!) output package format is Aab for convenience, as there is currently no way to select the gradle backend via command line parameters and hardcoding it in manifest.yaml makes it inconvenient to build Apks with the "native" builtin backend.

Also insert validation in case the user explicitly inserts gradle: false but tries to build Aabs. The default for releases is still Aab if there is an explicit gradle: true, but an Apk is built otherwise.

Instead of explicitly requiring `gradle: true` in the manifest, enable
it by default when the (implicit or explicit!) output package format
is `Aab` for convenience, as there is currently no way to select the
`gradle` backend via command line parameters and hardcoding it in
`manifest.yaml` makes it inconvenient to build `Apk`s with the "native"
builtin backend.

Also insert validation in case the user explicitly inserts `gradle:
false` but tries to build `Aab`s.  The default for releases is still
`Aab` if there is an explicit `gradle: true`, but an `Apk` is built
otherwise.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant