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

Commit bdad6c37 authored by Jooyung Han's avatar Jooyung Han Committed by Gerrit Code Review
Browse files

Merge "apex: __ANDROID_APEX__ defined with no value"

parents 9f221955 6b8459be

apex/apex_test.go

+6 −9
Original line number Diff line number Diff line
@@ -1212,24 +1212,21 @@ func TestMacro(t *testing.T) { 
		}

	`)



	// non-APEX variant does not have __ANDROID__APEX__ defined

	// non-APEX variant does not have __ANDROID_APEX(_NAME)__ defined

	mylibCFlags := ctx.ModuleForTests("mylib", "android_arm64_armv8-a_core_static").Rule("cc").Args["cFlags"]

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX__=myapex")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX__=otherapex")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX__")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX_MYAPEX__")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX_OTHERAPEX__")



	// APEX variant has __ANDROID_APEX__=<apexname> defined

	// APEX variant has __ANDROID_APEX(_NAME)__ defined

	mylibCFlags = ctx.ModuleForTests("mylib", "android_arm64_armv8-a_core_static_myapex").Rule("cc").Args["cFlags"]

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX__=myapex")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX__=otherapex")

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX__")

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX_MYAPEX__")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX_OTHERAPEX__")



	// APEX variant has __ANDROID_APEX__=<apexname> defined

	// APEX variant has __ANDROID_APEX(_NAME)__ defined

	mylibCFlags = ctx.ModuleForTests("mylib", "android_arm64_armv8-a_core_static_otherapex").Rule("cc").Args["cFlags"]

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX__=myapex")

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX__=otherapex")

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX__")

	ensureNotContains(t, mylibCFlags, "-D__ANDROID_APEX_MYAPEX__")

	ensureContains(t, mylibCFlags, "-D__ANDROID_APEX_OTHERAPEX__")

}

cc/compiler.go

+1 −2
Original line number Diff line number Diff line
@@ -320,8 +320,7 @@ func (compiler *baseCompiler) compilerFlags(ctx ModuleContext, flags Flags, deps 
	}



	if ctx.apexName() != "" {

		// TODO(b/142582178): remove the value for __ANDROID_APEX__

		flags.GlobalFlags = append(flags.GlobalFlags, "-D__ANDROID_APEX__="+ctx.apexName())

		flags.GlobalFlags = append(flags.GlobalFlags, "-D__ANDROID_APEX__")

		flags.GlobalFlags = append(flags.GlobalFlags, "-D__ANDROID_APEX_"+makeDefineString(ctx.apexName())+"__")

	}