-
-
Notifications
You must be signed in to change notification settings - Fork 23.9k
Deprecate the Android studio dev buildtype
#113469
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
base: master
Are you sure you want to change the base?
Conversation
Alex2782
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you also want to update the documentation (TODOs) in this PR?
At first glance, everything looks okay to me; the Dev builds are no longer listed.
./gradlew tasks --all | grep Dev
app:deviceAndroidTest - Installs and runs instrumentation tests using all Device Providers.
editor:deviceAndroidTest - Installs and runs instrumentation tests using all Device Providers.
lib:deviceAndroidTest - Installs and runs instrumentation tests using all Device Providers.
nativeSrcsConfigs:deviceAndroidTest - Installs and runs instrumentation tests using all Device Providers.
app:deviceCheck - Runs all device checks using Device Providers and Test Servers.
editor:deviceCheck - Runs all device checks using Device Providers and Test Servers.
lib:deviceCheck - Runs all device checks using Device Providers and Test Servers.
nativeSrcsConfigs:deviceCheck - Runs all device checks using Device Providers and Test Servers.
No the documentation will be updated in their respective PRs against the |
Is it strictly when building through Android Studio? I'm a bit confused by what this all actually means. I would expect the following (simplified) commands to produce a distinct, valid and usable output and not interfere at all with eachother, as is the case for the desktop platforms: scons target=editor # TOOLS_ENABLED, DEBUG_ENABLED
scons target=editor dev_build=yes # TOOLS_ENABLED, DEBUG_ENABLED, DEV_ENABLED
scons target=template_debug # DEBUG_ENABLED
scons target=template_debug dev_build=yes # DEBUG_ENABLED, DEV_ENABLED
scons target=template_release # N/A
scons target=template_release dev_build=yes # DEV_ENABLED(I have no idea how the Android editor works, so that part might not be applicable here at all.) Based on your comment I get the sense that a non-dev debug build (i.e. |
Yes it's strictly when building through Android Studio. When doing so, a custom When not building through Android Studio, things work as you described. |
The project is setup in Android Studio with three buildtypes: - `release` for release builds of the engine - `debug` for debug builds of the engine with `dev_mode`, `dev_build`, and `debug_symbols` disabled - `dev` for debug builds of the engine with `dev_mode`, `dev_build`, and `debug_symbols` enabled This commit deprecates and removes the `dev` buildtype and instead enables `dev_mode`, `dev_build`, and `debug_symbols` for the `debug` buildtype when building with Android Studio. The `release` buildtype has also been updated such that a `release` build built with Android Studio and signed with non-production keys can be installed side-by-side with a production-signed release (e.g: from the store).
d82356b to
571a723
Compare
The project is setup in Android Studio with three buildtypes:
releasefor release builds of the enginedebugfor debug builds of the engine withdev_mode,dev_build, anddebug_symbolsdisableddevfor debug builds of the engine withdev_mode,dev_build, anddebug_symbolsenabledThis PR deprecates and removes the
devbuildtype and instead enablesdev_mode,dev_build, anddebug_symbolsfor thedebugbuildtype when building with Android Studio.The
releasebuildtype has also been updated such that areleasebuild built with Android Studio and signed with non-production keys can be installed side-by-side with a production-signed release (e.g: from the store).TODO: