diff options
| author | 2023-10-26 16:07:29 +0000 | |
|---|---|---|
| committer | 2023-11-08 11:20:28 -0700 | |
| commit | 24366c842b903c29c39d36af42384e14573e9124 (patch) | |
| tree | e20995685ee9a91e4d0366aea75eef2a2f11c873 /tools/aconfig/src/codegen_java.rs | |
| parent | bcaac55c355ab1632223977e87bdb981c434042d (diff) | |
Require TARGET_RELEASE for builds
Rather than use an unsupported flag setting that the user likely
doesn't even realize is being used, we immediately stop the build.
This error message is more verbose, mentioning 'lunch', because
it's anticipated a lot more users will hit this issue when first
switching to trunk stable, and more details will hopefully help
them out.
We have some complication in that some internal commands set
TARGET_RELEASE to an empty string. We put in logic to allow
that path.
Since $(error) immediately stops the build, we also get rid of
some 'else' logic and indentation, to hopefully offset some of
the complication we've added.
Bug: 307946156
Test: 'lunch' (still) works; A build attempt without `TARGET_RELEASE` set (now) fails
Change-Id: I2f667668c6688e501a3536981b7bae5fdbf962a0
Diffstat (limited to 'tools/aconfig/src/codegen_java.rs')
0 files changed, 0 insertions, 0 deletions