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

Commit 8aa5b390 authored by Inseob Kim's avatar Inseob Kim Committed by Automerger Merge Worker
Browse files

Refine logic choosing vendor snapshot modules am: 7f283f4b 

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

Change-Id: Ic2926232e9ffff86146f2bdde8f589752eb01876
parents 4ba565ea 7f283f4b

cc/binary.go

+4 −0
Original line number Diff line number Diff line
@@ -231,6 +231,10 @@ func (binary *binaryDecorator) staticBinary() bool { 
	return binary.static()

}



func (binary *binaryDecorator) binary() bool {

	return true

}



func (binary *binaryDecorator) linkerFlags(ctx ModuleContext, flags Flags) Flags {

	flags = binary.baseLinker.linkerFlags(ctx, flags)

cc/cc.go

+15 −1
Original line number Diff line number Diff line
@@ -316,6 +316,7 @@ type ModuleContextIntf interface { 
	static() bool

	staticBinary() bool

	header() bool

	binary() bool

	toolchain() config.Toolchain

	canUseSdk() bool

	useSdk() bool
@@ -1129,6 +1130,10 @@ func (ctx *moduleContextImpl) header() bool { 
	return ctx.mod.header()

}



func (ctx *moduleContextImpl) binary() bool {

	return ctx.mod.binary()

}



func (ctx *moduleContextImpl) canUseSdk() bool {

	return ctx.mod.canUseSdk()

}
@@ -1925,7 +1930,7 @@ func (c *Module) DepsMutator(actx android.BottomUpMutatorContext) { 
	if deps.StaticUnwinderIfLegacy {

		actx.AddVariationDependencies([]blueprint.Variation{

			{Mutator: "link", Variation: "static"},

		}, staticUnwinderDepTag, staticUnwinder(actx))

		}, staticUnwinderDepTag, rewriteSnapshotLibs(staticUnwinder(actx), vendorSnapshotStaticLibs))

	}



	for _, lib := range deps.LateStaticLibs {
@@ -2752,6 +2757,15 @@ func (c *Module) header() bool { 
	return false

}



func (c *Module) binary() bool {

	if b, ok := c.linker.(interface {

		binary() bool

	}); ok {

		return b.binary()

	}

	return false

}



func (c *Module) getMakeLinkType(actx android.ModuleContext) string {

	if c.UseVndk() {

		if lib, ok := c.linker.(*llndkStubDecorator); ok {

cc/cc_test.go

+53 −21
Original line number Diff line number Diff line
@@ -258,9 +258,7 @@ func checkVndkModule(t *testing.T, ctx *android.TestContext, name, subDir string 
	}

}



func checkSnapshot(t *testing.T, ctx *android.TestContext, singletonName, moduleName, snapshotFilename, subDir, variant string) {

	snapshotSingleton := ctx.SingletonForTests(singletonName)



func checkSnapshot(t *testing.T, ctx *android.TestContext, singleton android.TestingSingleton, moduleName, snapshotFilename, subDir, variant string) {

	mod, ok := ctx.ModuleForTests(moduleName, variant).Module().(android.OutputFileProducer)

	if !ok {

		t.Errorf("%q must have output\n", moduleName)
@@ -273,7 +271,7 @@ func checkSnapshot(t *testing.T, ctx *android.TestContext, singletonName, module 
	}

	snapshotPath := filepath.Join(subDir, snapshotFilename)



	out := snapshotSingleton.Output(snapshotPath)

	out := singleton.Output(snapshotPath)

	if out.Input.String() != outputFiles[0].String() {

		t.Errorf("The input of snapshot %q must be %q, but %q", moduleName, out.Input.String(), outputFiles[0])

	}
@@ -398,16 +396,18 @@ func TestVndk(t *testing.T) { 
	variant := "android_vendor.VER_arm64_armv8-a_shared"

	variant2nd := "android_vendor.VER_arm_armv7-a-neon_shared"



	checkSnapshot(t, ctx, "vndk-snapshot", "libvndk", "libvndk.so", vndkCoreLibPath, variant)

	checkSnapshot(t, ctx, "vndk-snapshot", "libvndk", "libvndk.so", vndkCoreLib2ndPath, variant2nd)

	checkSnapshot(t, ctx, "vndk-snapshot", "libvndk_sp", "libvndk_sp-x.so", vndkSpLibPath, variant)

	checkSnapshot(t, ctx, "vndk-snapshot", "libvndk_sp", "libvndk_sp-x.so", vndkSpLib2ndPath, variant2nd)

	snapshotSingleton := ctx.SingletonForTests("vndk-snapshot")



	checkSnapshot(t, ctx, snapshotSingleton, "libvndk", "libvndk.so", vndkCoreLibPath, variant)

	checkSnapshot(t, ctx, snapshotSingleton, "libvndk", "libvndk.so", vndkCoreLib2ndPath, variant2nd)

	checkSnapshot(t, ctx, snapshotSingleton, "libvndk_sp", "libvndk_sp-x.so", vndkSpLibPath, variant)

	checkSnapshot(t, ctx, snapshotSingleton, "libvndk_sp", "libvndk_sp-x.so", vndkSpLib2ndPath, variant2nd)



	snapshotConfigsPath := filepath.Join(snapshotVariantPath, "configs")

	checkSnapshot(t, ctx, "vndk-snapshot", "llndk.libraries.txt", "llndk.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, "vndk-snapshot", "vndkcore.libraries.txt", "vndkcore.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, "vndk-snapshot", "vndksp.libraries.txt", "vndksp.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, "vndk-snapshot", "vndkprivate.libraries.txt", "vndkprivate.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, snapshotSingleton, "llndk.libraries.txt", "llndk.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, snapshotSingleton, "vndkcore.libraries.txt", "vndkcore.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, snapshotSingleton, "vndksp.libraries.txt", "vndksp.libraries.txt", snapshotConfigsPath, "")

	checkSnapshot(t, ctx, snapshotSingleton, "vndkprivate.libraries.txt", "vndkprivate.libraries.txt", snapshotConfigsPath, "")



	checkVndkOutput(t, ctx, "vndk/vndk.libraries.txt", []string{

		"LLNDK: libc.so",
@@ -839,6 +839,12 @@ func TestVendorSnapshot(t *testing.T) { 
		vendor_available: true,

		nocrt: true,

	}



	toolchain_library {

		name: "libb",

		vendor_available: true,

		src: "libb.a",

	}

`

	config := TestConfig(buildDir, android.Android, nil, bp, nil)

	config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
@@ -849,6 +855,9 @@ func TestVendorSnapshot(t *testing.T) { 


	snapshotDir := "vendor-snapshot"

	snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64")

	snapshotSingleton := ctx.SingletonForTests("vendor-snapshot")



	var jsonFiles []string



	for _, arch := range [][]string{

		[]string{"arm64", "armv8-a"},
@@ -861,22 +870,45 @@ func TestVendorSnapshot(t *testing.T) { 
		// For shared libraries, only non-VNDK vendor_available modules are captured

		sharedVariant := fmt.Sprintf("android_vendor.VER_%s_%s_shared", archType, archVariant)

		sharedDir := filepath.Join(snapshotVariantPath, archDir, "shared")

		checkSnapshot(t, ctx, "vendor-snapshot", "libvendor", "libvendor.so", sharedDir, sharedVariant)

		checkSnapshot(t, ctx, "vendor-snapshot", "libvendor_available", "libvendor_available.so", sharedDir, sharedVariant)

		checkSnapshot(t, ctx, snapshotSingleton, "libvendor", "libvendor.so", sharedDir, sharedVariant)

		checkSnapshot(t, ctx, snapshotSingleton, "libvendor_available", "libvendor_available.so", sharedDir, sharedVariant)

		jsonFiles = append(jsonFiles,

			filepath.Join(sharedDir, "libvendor.so.json"),

			filepath.Join(sharedDir, "libvendor_available.so.json"))



		// For static libraries, all vendor:true and vendor_available modules (including VNDK) are captured.

		staticVariant := fmt.Sprintf("android_vendor.VER_%s_%s_static", archType, archVariant)

		staticDir := filepath.Join(snapshotVariantPath, archDir, "static")

		checkSnapshot(t, ctx, "vendor-snapshot", "libvndk", "libvndk.a", staticDir, staticVariant)

		checkSnapshot(t, ctx, "vendor-snapshot", "libvendor", "libvendor.a", staticDir, staticVariant)

		checkSnapshot(t, ctx, "vendor-snapshot", "libvendor_available", "libvendor_available.a", staticDir, staticVariant)



		// For binary libraries, all vendor:true and vendor_available modules are captured.

		checkSnapshot(t, ctx, snapshotSingleton, "libb", "libb.a", staticDir, staticVariant)

		checkSnapshot(t, ctx, snapshotSingleton, "libvndk", "libvndk.a", staticDir, staticVariant)

		checkSnapshot(t, ctx, snapshotSingleton, "libvendor", "libvendor.a", staticDir, staticVariant)

		checkSnapshot(t, ctx, snapshotSingleton, "libvendor_available", "libvendor_available.a", staticDir, staticVariant)

		jsonFiles = append(jsonFiles,

			filepath.Join(staticDir, "libb.a.json"),

			filepath.Join(staticDir, "libvndk.a.json"),

			filepath.Join(staticDir, "libvendor.a.json"),

			filepath.Join(staticDir, "libvendor_available.a.json"))



		// For binary executables, all vendor:true and vendor_available modules are captured.

		if archType == "arm64" {

			binaryVariant := fmt.Sprintf("android_vendor.VER_%s_%s", archType, archVariant)

			binaryDir := filepath.Join(snapshotVariantPath, archDir, "binary")

			checkSnapshot(t, ctx, "vendor-snapshot", "vendor_bin", "vendor_bin", binaryDir, binaryVariant)

			checkSnapshot(t, ctx, "vendor-snapshot", "vendor_available_bin", "vendor_available_bin", binaryDir, binaryVariant)

			checkSnapshot(t, ctx, snapshotSingleton, "vendor_bin", "vendor_bin", binaryDir, binaryVariant)

			checkSnapshot(t, ctx, snapshotSingleton, "vendor_available_bin", "vendor_available_bin", binaryDir, binaryVariant)

			jsonFiles = append(jsonFiles,

				filepath.Join(binaryDir, "vendor_bin.json"),

				filepath.Join(binaryDir, "vendor_available_bin.json"))

		}



		// For header libraries, all vendor:true and vendor_available modules are captured.

		headerDir := filepath.Join(snapshotVariantPath, archDir, "header")

		jsonFiles = append(jsonFiles, filepath.Join(headerDir, "libvendor_headers.json"))

	}



	for _, jsonFile := range jsonFiles {

		// verify all json files exist

		if snapshotSingleton.MaybeOutput(jsonFile).Rule == nil {

			t.Errorf("%q expected but not found", jsonFile)

		}

	}

}

cc/prebuilt.go

+4 −0
Original line number Diff line number Diff line
@@ -349,6 +349,10 @@ func (p *prebuiltBinaryLinker) link(ctx ModuleContext, 
	return nil

}



func (p *prebuiltBinaryLinker) binary() bool {

	return true

}



// cc_prebuilt_binary installs a precompiled executable in srcs property in the

// device's directory.

func prebuiltBinaryFactory() android.Module {

cc/testing.go

+1 −0
Original line number Diff line number Diff line
@@ -496,6 +496,7 @@ func TestConfig(buildDir string, os android.OsType, env map[string]string, 
		"my_include":  nil,

		"foo.map.txt": nil,

		"liba.so":     nil,

		"libb.a":      nil,

	}



	GatherRequiredFilesForTest(mockFS)