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

Commit 3affc07c authored by Jaewoong Jung's avatar Jaewoong Jung
Browse files

Revert "Export static libraries." 

This reverts commit 9d5ca159.

Reason for revert: This broke few builds.

Change-Id: Id9c4d7c145602a55ae07bb177f2c236e16f4fb63
parent 9d5ca159

Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -435,7 +435,6 @@ toolchain_library { 
            src: "prebuilts/gcc/linux-x86/host/x86_64-w64-mingw32-4.8/x86_64-w64-mingw32/lib/libwinpthread.a",

        },

    },

    notice: "../../prebuilts/gcc/linux-x86/host/x86_64-w64-mingw32-4.8/licenses/mingw-w64-svn-r5861/mingw-w64-libraries/winpthreads/COPYING",

}



toolchain_library {

android/androidmk.go

+3 −4
Original line number Diff line number Diff line
@@ -275,11 +275,10 @@ func translateAndroidModule(ctx SingletonContext, w io.Writer, mod blueprint.Mod 
		if amod.commonProperties.Owner != nil {

			fmt.Fprintln(&data.preamble, "LOCAL_MODULE_OWNER :=", *amod.commonProperties.Owner)

		}

	}



		if amod.commonProperties.Notice != nil {

			fmt.Fprintln(&data.preamble, "LOCAL_NOTICE_FILE :=", "$(LOCAL_PATH)/"+*amod.commonProperties.Notice)

		}

	}



	if host {

		makeOs := amod.Os().String()

cc/androidmk.go

+0 −6
Original line number Diff line number Diff line
@@ -70,12 +70,6 @@ func (c *Module) AndroidMk() android.AndroidMkData { 
				if len(c.Properties.AndroidMkSharedLibs) > 0 {

					fmt.Fprintln(w, "LOCAL_SHARED_LIBRARIES := "+strings.Join(c.Properties.AndroidMkSharedLibs, " "))

				}

				if len(c.Properties.AndroidMkStaticLibs) > 0 {

					fmt.Fprintln(w, "LOCAL_STATIC_LIBRARIES := "+strings.Join(c.Properties.AndroidMkStaticLibs, " "))

				}

				if len(c.Properties.AndroidMkWholeStaticLibs) > 0 {

					fmt.Fprintln(w, "LOCAL_WHOLE_STATIC_LIBRARIES := "+strings.Join(c.Properties.AndroidMkWholeStaticLibs, " "))

				}

				fmt.Fprintln(w, "LOCAL_SOONG_LINK_TYPE :=", c.getMakeLinkType())

				if c.useVndk() {

					fmt.Fprintln(w, "LOCAL_USE_VNDK := true")

cc/cc.go

+4 −12
Original line number Diff line number Diff line
@@ -177,9 +177,7 @@ type BaseProperties struct { 
	Sdk_version *string



	AndroidMkSharedLibs  []string `blueprint:"mutated"`

	AndroidMkStaticLibs      []string `blueprint:"mutated"`

	AndroidMkRuntimeLibs []string `blueprint:"mutated"`

	AndroidMkWholeStaticLibs []string `blueprint:"mutated"`

	HideFromMake         bool     `blueprint:"mutated"`

	PreventInstall       bool     `blueprint:"mutated"`
@@ -1481,15 +1479,9 @@ func (c *Module) depsToPaths(ctx android.ModuleContext) PathDeps { 
			// they merely serve as Make dependencies and do not affect this lib itself.

			c.Properties.AndroidMkSharedLibs = append(

				c.Properties.AndroidMkSharedLibs, makeLibName(depName))

		case staticDepTag, staticExportDepTag, lateStaticDepTag:

			c.Properties.AndroidMkStaticLibs = append(

				c.Properties.AndroidMkStaticLibs, makeLibName(depName))

		case runtimeDepTag:

			c.Properties.AndroidMkRuntimeLibs = append(

				c.Properties.AndroidMkRuntimeLibs, makeLibName(depName))

		case wholeStaticDepTag:

			c.Properties.AndroidMkWholeStaticLibs = append(

				c.Properties.AndroidMkWholeStaticLibs, makeLibName(depName))

		}

	})

cc/cc_test.go

+0 −37
Original line number Diff line number Diff line
@@ -1549,43 +1549,6 @@ func TestRuntimeLibsNoVndk(t *testing.T) { 
	checkRuntimeLibs(t, []string{"libvendor_available1", "libvendor1"}, module)

}



func checkStaticLibs(t *testing.T, expected []string, module *Module) {

	actual := module.Properties.AndroidMkStaticLibs

	if !reflect.DeepEqual(actual, expected) {

		t.Errorf("incorrect static_libs"+

			"\nactual:   %v"+

			"\nexpected: %v",

			actual,

			expected,

		)

	}

}



const staticLibAndroidBp = `

	cc_library {

		name: "lib1",

	}

	cc_library {

		name: "lib2",

		static_libs: ["lib1"],

	}

`



func TestStaticLibDepExport(t *testing.T) {

	ctx := testCc(t, staticLibAndroidBp)



	// Check the shared version of lib2.

	variant := "android_arm64_armv8-a_core_shared"

	module := ctx.ModuleForTests("lib2", variant).Module().(*Module)

	checkStaticLibs(t, []string{"lib1", "libclang_rt.builtins-aarch64-android", "libatomic", "libgcc"}, module)



	// Check the static version of lib2.

	variant = "android_arm64_armv8-a_core_static"

	module = ctx.ModuleForTests("lib2", variant).Module().(*Module)

	// libc++_static is linked additionally.

	checkStaticLibs(t, []string{"lib1", "libc++_static", "libclang_rt.builtins-aarch64-android", "libatomic", "libgcc"}, module)

}



var compilerFlagsTestCases = []struct {

	in  string

	out bool