Merge "Forbid -Weverything"
am: f3898e3bef
Change-Id: I998cfb29f235fcb289dcacd2a13f961e35a906c6
This commit is contained in:
@@ -38,6 +38,11 @@ func CheckBadCompilerFlags(ctx BaseModuleContext, prop string, flags []string) {
|
|||||||
ctx.PropertyErrorf(prop, "Illegal flag `%s`", flag)
|
ctx.PropertyErrorf(prop, "Illegal flag `%s`", flag)
|
||||||
} else if flag == "--coverage" {
|
} else if flag == "--coverage" {
|
||||||
ctx.PropertyErrorf(prop, "Bad flag: `%s`, use native_coverage instead", flag)
|
ctx.PropertyErrorf(prop, "Bad flag: `%s`, use native_coverage instead", flag)
|
||||||
|
} else if flag == "-Weverything" {
|
||||||
|
if !ctx.Config().IsEnvTrue("ANDROID_TEMPORARILY_ALLOW_WEVERYTHING") {
|
||||||
|
ctx.PropertyErrorf(prop, "-Weverything is not allowed in Android.bp files. "+
|
||||||
|
"Build with `m ANDROID_TEMPORARILY_ALLOW_WEVERYTHING=true` to experiment locally with -Weverything.")
|
||||||
|
}
|
||||||
} else if strings.Contains(flag, " ") {
|
} else if strings.Contains(flag, " ") {
|
||||||
args := strings.Split(flag, " ")
|
args := strings.Split(flag, " ")
|
||||||
if args[0] == "-include" {
|
if args[0] == "-include" {
|
||||||
|
Reference in New Issue
Block a user