Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 50a9771a authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Change default to require licenses property." am: 7bb02384 am:... 

Merge "Change default to require licenses property." am: 7bb02384 am: ec01d2d7 am: 4bb42048 am: 48f7e08e

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1818661

Change-Id: I4add2ffdd59f560411c50b49c693b6a812a7ae3f
parents fc497c09 48f7e08e

android/androidmk.go

+1 −1
Original line number Diff line number Diff line
@@ -845,7 +845,7 @@ func translateAndroidModule(ctx SingletonContext, w io.Writer, mod blueprint.Mod 
		case "*selinux.selinuxContextsModule": // license properties written

		case "*sysprop.syspropLibrary": // license properties written

		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))

			}

		}

android/licenses.go

+1 −1
Original line number Diff line number Diff line
@@ -253,7 +253,7 @@ func getLicenses(ctx BaseModuleContext, module Module) []string { 


	primaryProperty := module.base().primaryLicensesProperty

	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))

		}

		return nil