StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Change default to require licenses property."

Browse Source
This commit is contained in:
Treehugger Robot
2021-09-13 19:25:57 +00:00
committed by Gerrit Code Review
parent 861441fa36 65ee90a362
commit 7bb0238418
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
android/androidmk.go
View File

@@ -845,7 +845,7 @@ func translateAndroidModule(ctx SingletonContext, w io.Writer, mod blueprint.Mod
case "*selinux.selinuxContextsModule": // license properties written case "*selinux.selinuxContextsModule": // license properties written
case "*sysprop.syspropLibrary": // license properties written case "*sysprop.syspropLibrary": // license properties written
default: default:
if ctx.Config().IsEnvTrue("ANDROID_REQUIRE_LICENSES") { if !ctx.Config().IsEnvFalse("ANDROID_REQUIRE_LICENSES") {
return fmt.Errorf("custom make rules not allowed for %q (%q) module %q", ctx.ModuleType(mod), reflect.TypeOf(mod), ctx.ModuleName(mod)) return fmt.Errorf("custom make rules not allowed for %q (%q) module %q", ctx.ModuleType(mod), reflect.TypeOf(mod), ctx.ModuleName(mod))
} }
} }

2
android/licenses.go
View File

@@ -253,7 +253,7 @@ func getLicenses(ctx BaseModuleContext, module Module) []string {
primaryProperty := module.base().primaryLicensesProperty primaryProperty := module.base().primaryLicensesProperty
if primaryProperty == nil { if primaryProperty == nil {
if ctx.Config().IsEnvTrue("ANDROID_REQUIRE_LICENSES") { if !ctx.Config().IsEnvFalse("ANDROID_REQUIRE_LICENSES") {
ctx.ModuleErrorf("module type %q must have an applicable licenses property", ctx.OtherModuleType(module)) ctx.ModuleErrorf("module type %q must have an applicable licenses property", ctx.OtherModuleType(module))
} }
return nil return nil