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

Commit 68e126ee authored by Wilson Sung's avatar Wilson Sung Committed by Gerrit Code Review
Browse files

Revert "Produce _alwayslink sibling targets through bp2build" 

Revert submission 2467901-b/267214124

Reason for revert: compile error

Reverted changes: /q/submissionid:2467901-b/267214124

Change-Id: I2235fd1457fda60b4613d5512f7fb20b7044a2db
Fix: 271943047
parent 011b7cf0

bp2build/cc_prebuilt_library_conversion_test.go

+2 −26
Original line number Diff line number Diff line
@@ -39,10 +39,6 @@ cc_prebuilt_library { 
				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static", AttrNameToString{

					"static_library": `"libf.so"`,

				}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static_alwayslink", AttrNameToString{

					"static_library": `"libf.so"`,

					"alwayslink":     "True",

				}),

				MakeBazelTarget("cc_prebuilt_library_shared", "libtest", AttrNameToString{

					"shared_library": `"libf.so"`,

				}),
@@ -75,14 +71,8 @@ cc_prebuilt_library { 
        "//build/bazel/platforms/arch:arm": "libg.so",

        "//build/bazel/platforms/arch:arm64": "libf.so",

        "//conditions:default": None,

    })`}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static_alwayslink", AttrNameToString{

					"alwayslink": "True",

					"static_library": `select({

        "//build/bazel/platforms/arch:arm": "libg.so",

        "//build/bazel/platforms/arch:arm64": "libf.so",

        "//conditions:default": None,

    })`}),

    })`,

				}),

				MakeBazelTarget("cc_prebuilt_library_shared", "libtest", AttrNameToString{

					"shared_library": `select({

        "//build/bazel/platforms/arch:arm": "libg.so",
@@ -119,12 +109,6 @@ cc_prebuilt_library { 
					"export_includes":        `["testdir/1/"]`,

					"export_system_includes": `["testdir/2/"]`,

				}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static_alwayslink", AttrNameToString{

					"static_library":         `"libf.so"`,

					"export_includes":        `["testdir/1/"]`,

					"export_system_includes": `["testdir/2/"]`,

					"alwayslink":             "True",

				}),

				// TODO(b/229374533): When fixed, update this test

				MakeBazelTarget("cc_prebuilt_library_shared", "libtest", AttrNameToString{

					"shared_library": `"libf.so"`,
@@ -204,10 +188,6 @@ cc_prebuilt_library { 
				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static", AttrNameToString{

					"static_library": `"libf.so"`,

				}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static_alwayslink", AttrNameToString{

					"static_library": `"libf.so"`,

					"alwayslink":     "True",

				}),

				MakeBazelTarget("cc_prebuilt_library_shared", "libtest", AttrNameToString{

					"shared_library": `"libg.so"`,

				}),
@@ -265,10 +245,6 @@ cc_prebuilt_library { 
//				makeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static", attrNameToString{

//					"static_library": `"libf.so"`,

//				}),

//				makeBazelTarget("cc_prebuilt_library_static", "libtest_bp2build_cc_library_static_always", attrNameToString{

//					"static_library": `"libf.so"`,

//					"alwayslink": "True",

//				}),

//			},

//		})

//}

bp2build/cc_prebuilt_library_static_test.go

+2 −12
Original line number Diff line number Diff line
@@ -39,10 +39,6 @@ cc_prebuilt_library_static { 
				MakeBazelTarget("cc_prebuilt_library_static", "libtest", AttrNameToString{

					"static_library": `"libf.so"`,

				}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_alwayslink", AttrNameToString{

					"static_library": `"libf.so"`,

					"alwayslink":     "True",

				}),

			},

		})

}
@@ -72,14 +68,8 @@ cc_prebuilt_library_static { 
        "//build/bazel/platforms/arch:arm": "libg.so",

        "//build/bazel/platforms/arch:arm64": "libf.so",

        "//conditions:default": None,

    })`}),

				MakeBazelTarget("cc_prebuilt_library_static", "libtest_alwayslink", AttrNameToString{

					"alwayslink": "True",

					"static_library": `select({

        "//build/bazel/platforms/arch:arm": "libg.so",

        "//build/bazel/platforms/arch:arm64": "libf.so",

        "//conditions:default": None,

    })`}),

    })`,

				}),

			},

		})

}

cc/prebuilt.go

+0 −6
Original line number Diff line number Diff line
@@ -352,7 +352,6 @@ type bazelPrebuiltLibraryStaticAttributes struct { 
	Static_library         bazel.LabelAttribute

	Export_includes        bazel.StringListAttribute

	Export_system_includes bazel.StringListAttribute

	Alwayslink             bazel.BoolAttribute

}



// TODO(b/228623543): The below is not entirely true until the bug is fixed. For now, both targets are always generated
@@ -390,11 +389,6 @@ func prebuiltLibraryStaticBp2Build(ctx android.TopDownMutatorContext, module *Mo 


	tags := android.ApexAvailableTags(module)

	ctx.CreateBazelTargetModuleWithRestrictions(props, android.CommonAttributes{Name: name, Tags: tags}, attrs, prebuiltAttrs.Enabled)



	_true := true

	alwayslinkAttrs := *attrs

	alwayslinkAttrs.Alwayslink.SetValue(&_true)

	ctx.CreateBazelTargetModuleWithRestrictions(props, android.CommonAttributes{Name: name + "_alwayslink", Tags: tags}, &alwayslinkAttrs, prebuiltAttrs.Enabled)

}



type bazelPrebuiltLibrarySharedAttributes struct {