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

Commit b33e0b8b authored by Romain Jobredeaux's avatar Romain Jobredeaux
Browse files

Revert "Add static_libs support to android_app bp2build converter." 

This reverts commit de722acd.

Reason for revert: broke bp2build CI

Change-Id: I535e8746e846c20f1a47c2a1eb0362276af49206
parent de722acd

android/bazel.go

+0 −1
Original line number Diff line number Diff line
@@ -281,7 +281,6 @@ var ( 
		"development/samples/USB/MissileLauncher":            Bp2BuildDefaultTrue,

		"development/samples/VoiceRecognitionService":        Bp2BuildDefaultTrue,

		"development/samples/VoicemailProviderDemo":          Bp2BuildDefaultTrue,

		"development/samples/WiFiDirectDemo":                 Bp2BuildDefaultTrue,

		"development/sdk":                                    Bp2BuildDefaultTrueRecursively,

		"external/arm-optimized-routines":                    Bp2BuildDefaultTrueRecursively,

		"external/boringssl":                                 Bp2BuildDefaultTrueRecursively,

bp2build/android_app_conversion_test.go

+1 −3
Original line number Diff line number Diff line
@@ -66,7 +66,7 @@ func TestAndroidAppAllSupportedFields(t *testing.T) { 
			"resb/res.png":                 "",

			"manifest/AndroidManifest.xml": "",

		},

		blueprint: simpleModuleDoNotConvertBp2build("android_app", "static_lib_dep") + `

		blueprint: `

android_app {

        name: "TestApp",

        srcs: ["app.java"],
@@ -74,7 +74,6 @@ android_app { 
        package_name: "com.google",

        resource_dirs: ["resa", "resb"],

        manifest: "manifest/AndroidManifest.xml",

        static_libs: ["static_lib_dep"]

}

`,

		expectedBazelTargets: []string{
@@ -86,7 +85,6 @@ android_app { 
        "resb/res.png",

    ]`,

				"custom_package": `"com.google"`,

				"deps":           `[":static_lib_dep"]`,

			}),

		}})

}

java/app.go

+0 −4
Original line number Diff line number Diff line
@@ -1432,7 +1432,6 @@ type bazelAndroidAppAttributes struct { 
	Manifest       bazel.Label

	Custom_package *string

	Resource_files bazel.LabelListAttribute

	Deps           bazel.LabelListAttribute

}



// ConvertWithBp2build is used to convert android_app to Bazel.
@@ -1450,15 +1449,12 @@ func (a *AndroidApp) ConvertWithBp2build(ctx android.TopDownMutatorContext) { 
		resourceFiles.Includes = append(resourceFiles.Includes, files...)

	}



	deps := bazel.MakeLabelListAttribute(android.BazelLabelForModuleDeps(ctx, a.properties.Static_libs))



	attrs := &bazelAndroidAppAttributes{

		Srcs:     srcs,

		Manifest: android.BazelLabelForModuleSrcSingle(ctx, manifest),

		// TODO(b/209576404): handle package name override by product variable PRODUCT_MANIFEST_PACKAGE_NAME_OVERRIDES

		Custom_package: a.overridableAppProperties.Package_name,

		Resource_files: bazel.MakeLabelListAttribute(resourceFiles),

		Deps:           deps,

	}

	props := bazel.BazelTargetModuleProperties{Rule_class: "android_binary",

		Bzl_load_location: "@rules_android//rules:rules.bzl"}