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

Commit bd9bd6b2 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge changes from topic "import-sdk-version-none" into main am: a815d9bf 

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



Change-Id: I0ffdd8866c532d43d111e49e28c84c637f39eee7
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents aca52691 a815d9bf

bp2build/java_import_conversion_test.go

+29 −21
Original line number Diff line number Diff line
@@ -52,13 +52,15 @@ java_import { 
}

`,

		ExpectedBazelTargets: []string{

			MakeBazelTarget("filegroup", "example_import-jars", AttrNameToString{

				"srcs": `["import.jar"]`,

			}),

			MakeBazelTarget("java_import", "example_import", AttrNameToString{

				"jars": `["import.jar"]`,

				"jars": `[":example_import-jars"]`,

			}),

			MakeBazelTarget("java_library", "example_import-neverlink", AttrNameToString{

				"exports":     `[":example_import"]`,

			MakeBazelTarget("java_import", "example_import-neverlink", AttrNameToString{

				"jars":      `[":example_import-jars"]`,

				"neverlink": `True`,

				"sdk_version": `"none"`,

			}),

		}})

}
@@ -86,17 +88,19 @@ java_import { 
}

`,

		ExpectedBazelTargets: []string{

			MakeBazelTarget("java_import", "example_import", AttrNameToString{

				"jars": `select({

			MakeBazelTarget("filegroup", "example_import-jars", AttrNameToString{

				"srcs": `select({

        "//build/bazel_common_rules/platforms/os:android": ["android.jar"],

        "//build/bazel_common_rules/platforms/os:linux_glibc": ["linux.jar"],

        "//conditions:default": [],

    })`,

			}),

			MakeBazelTarget("java_library", "example_import-neverlink", AttrNameToString{

				"exports":     `[":example_import"]`,

			MakeBazelTarget("java_import", "example_import", AttrNameToString{

				"jars": `[":example_import-jars"]`,

			}),

			MakeBazelTarget("java_import", "example_import-neverlink", AttrNameToString{

				"jars":      `[":example_import-jars"]`,

				"neverlink": `True`,

				"sdk_version": `"none"`,

			}),

		}})

}
@@ -117,13 +121,15 @@ java_import_host { 
}

`,

		ExpectedBazelTargets: []string{

			MakeBazelTarget("filegroup", "example_import-jars", AttrNameToString{

				"srcs": `["import.jar"]`,

			}),

			MakeBazelTarget("java_import", "example_import", AttrNameToString{

				"jars": `["import.jar"]`,

				"jars": `[":example_import-jars"]`,

			}),

			MakeBazelTarget("java_library", "example_import-neverlink", AttrNameToString{

				"exports":     `[":example_import"]`,

			MakeBazelTarget("java_import", "example_import-neverlink", AttrNameToString{

				"jars":      `[":example_import-jars"]`,

				"neverlink": `True`,

				"sdk_version": `"none"`,

			}),

		}})

}
@@ -142,13 +148,15 @@ func TestJavaImportSameNameAsJavaLibrary(t *testing.T) { 
}

`,

		ExpectedBazelTargets: []string{

			MakeBazelTarget("filegroup", "test_lib-jars", AttrNameToString{

				"srcs": `["test.jar"]`,

			}),

			MakeBazelTarget("java_import", "test_lib", AttrNameToString{

				"jars": `["test.jar"]`,

				"jars": `[":test_lib-jars"]`,

			}),

			MakeBazelTarget("java_library", "test_lib-neverlink", AttrNameToString{

				"exports":     `[":test_lib"]`,

			MakeBazelTarget("java_import", "test_lib-neverlink", AttrNameToString{

				"jars":      `[":test_lib-jars"]`,

				"neverlink": `True`,

				"sdk_version": `"none"`,

			}),

		},

	}, func(ctx android.RegistrationContext) {

java/java.go

+32 −15
Original line number Diff line number Diff line
@@ -2970,6 +2970,10 @@ func (m *Library) convertLibraryAttrsBp2Build(ctx android.Bp2buildMutatorContext 
		// TODO(b/297356582): handle core_platform in bp2build

		ctx.MarkBp2buildUnconvertible(bp2build_metrics_proto.UnconvertedReasonType_PROPERTY_UNSUPPORTED, "sdk_version core_platform")

		return &javaCommonAttributes{}, &bp2BuildJavaInfo{}, false

	} else if proptools.String(m.deviceProperties.Sdk_version) == "none" {

		// TODO(b/297356703): handle system_modules

		ctx.MarkBp2buildUnconvertible(bp2build_metrics_proto.UnconvertedReasonType_PROPERTY_UNSUPPORTED, "sdk_version none")

		return &javaCommonAttributes{}, &bp2BuildJavaInfo{}, false

	}



	archVariantProps := m.GetArchVariantProperties(ctx, &CommonProperties{})
@@ -3403,9 +3407,14 @@ func createLibraryTarget(ctx android.Bp2buildMutatorContext, libInfo libraryCrea 
	return libName

}



type bazelJavaImportAttributes struct {

type importAttributes struct {

	Jars      bazel.LabelListAttribute

	Exports   bazel.LabelListAttribute

	Neverlink *bool

}



type filegroupAttrs struct {

	Srcs bazel.LabelListAttribute

}



// java_import bp2Build converter.
@@ -3421,28 +3430,36 @@ func (i *Import) ConvertWithBp2build(ctx android.Bp2buildMutatorContext) { 
		}

	}



	attrs := &bazelJavaImportAttributes{

		Jars: jars,

	name := android.RemoveOptionalPrebuiltPrefix(i.Name())

	filegroupTargetName := name + "-jars"



	ctx.CreateBazelTargetModule(

		bazel.BazelTargetModuleProperties{

			Rule_class:        "filegroup",

			Bzl_load_location: "//build/bazel/rules:filegroup.bzl",

		},

		android.CommonAttributes{Name: filegroupTargetName},

		&filegroupAttrs{

			Srcs: jars,

		},

	)



	attrs := &importAttributes{

		Jars: bazel.MakeSingleLabelListAttribute(bazel.Label{Label: ":" + filegroupTargetName}),

	}

	props := bazel.BazelTargetModuleProperties{

		Rule_class:        "java_import",

		Bzl_load_location: "//build/bazel/rules/java:import.bzl",

	}



	name := android.RemoveOptionalPrebuiltPrefix(i.Name())



	ctx.CreateBazelTargetModule(props, android.CommonAttributes{Name: name}, attrs)



	neverlink := true

	neverlinkAttrs := &javaLibraryAttributes{

		Neverlink: bazel.BoolAttribute{Value: &neverlink},

		Exports:   bazel.MakeSingleLabelListAttribute(bazel.Label{Label: ":" + name}),

		javaCommonAttributes: &javaCommonAttributes{

			Sdk_version: bazel.StringAttribute{Value: proptools.StringPtr("none")},

		},

	neverlinkAttrs := &importAttributes{

		Jars:      attrs.Jars,

		Neverlink: proptools.BoolPtr(true),

	}

	ctx.CreateBazelTargetModule(

		javaLibraryBazelTargetModuleProperties(),

		props,

		android.CommonAttributes{Name: name + "-neverlink"},

		neverlinkAttrs)

}