Merge changes from topic "kotlin_kythe" into main (6556d043) · Commits · e / os / android_build_soong

build_kzip.bash

+1 −0

Original line number	Diff line number	Diff line
		@@ -40,6 +40,7 @@ kzip_targets=(
		merge_zips
		xref_cxx
		xref_java
		xref_kotlin
		# TODO: b/286390153 - reenable rust
		# xref_rust
		)

java/base.go

+3 −2

Original line number	Diff line number	Diff line
		@@ -536,6 +536,7 @@ type Module struct {

		// list of the xref extraction files
		kytheFiles android.Paths
		kytheKotlinFiles android.Paths

		hideApexVariantFromMake bool

		@@ -1370,7 +1371,7 @@ func (j *Module) compile(ctx android.ModuleContext, extraSrcJars, extraClasspath

		kotlinJar := android.PathForModuleOut(ctx, "kotlin", jarName)
		kotlinHeaderJar := android.PathForModuleOut(ctx, "kotlin_headers", jarName)
		kotlinCompile(ctx, kotlinJar, kotlinHeaderJar, uniqueSrcFiles, kotlinCommonSrcFiles, srcJars, flags)
		j.kotlinCompile(ctx, kotlinJar, kotlinHeaderJar, uniqueSrcFiles, kotlinCommonSrcFiles, srcJars, flags)
		if ctx.Failed() {
		return
		}

java/config/config.go

+1 −0

Original line number	Diff line number	Diff line
		@@ -145,6 +145,7 @@ func init() {
		pctx.SourcePathVariable("JmodCmd", "${JavaToolchain}/jmod")
		pctx.SourcePathVariable("JrtFsJar", "${JavaHome}/lib/jrt-fs.jar")
		pctx.SourcePathVariable("JavaKytheExtractorJar", "prebuilts/build-tools/common/framework/javac_extractor.jar")
		pctx.SourcePathVariable("KotlinKytheExtractor", "prebuilts/build-tools/${hostPrebuiltTag}/bin/kotlinc_extractor")
		pctx.SourcePathVariable("Ziptime", "prebuilts/build-tools/${hostPrebuiltTag}/bin/ziptime")

		pctx.SourcePathVariable("ResourceProcessorBusyBox", "prebuilts/bazel/common/android_tools/android_tools/all_android_tools_deploy.jar")

java/java.go

+10 −0

Original line number	Diff line number	Diff line
		@@ -356,12 +356,17 @@ type UsesLibraryDependency interface {
		// TODO(jungjw): Move this to kythe.go once it's created.
		type xref interface {
		XrefJavaFiles() android.Paths
		XrefKotlinFiles() android.Paths
		}

		func (j *Module) XrefJavaFiles() android.Paths {
		return j.kytheFiles
		}

		func (j *Module) XrefKotlinFiles() android.Paths {
		return j.kytheKotlinFiles
		}

		func (d dependencyTag) PropagateAconfigValidation() bool {
		return d.static
		}
		@@ -3304,15 +3309,20 @@ type kytheExtractJavaSingleton struct {

		func (ks *kytheExtractJavaSingleton) GenerateBuildActions(ctx android.SingletonContext) {
		var xrefTargets android.Paths
		var xrefKotlinTargets android.Paths
		ctx.VisitAllModules(func(module android.Module) {
		if javaModule, ok := module.(xref); ok {
		xrefTargets = append(xrefTargets, javaModule.XrefJavaFiles()...)
		xrefKotlinTargets = append(xrefKotlinTargets, javaModule.XrefKotlinFiles()...)
		}
		})
		// TODO(asmundak): perhaps emit a rule to output a warning if there were no xrefTargets
		if len(xrefTargets) > 0 {
		ctx.Phony("xref_java", xrefTargets...)
		}
		if len(xrefKotlinTargets) > 0 {
		ctx.Phony("xref_kotlin", xrefKotlinTargets...)
		}
		}

		var Bool = proptools.Bool

java/kotlin.go

+49 −1

Original line number	Diff line number	Diff line
		@@ -64,6 +64,29 @@ var kotlinc = pctx.AndroidRemoteStaticRule("kotlinc", android.RemoteRuleSupports
		"kotlincFlags", "classpath", "srcJars", "commonSrcFilesArg", "srcJarDir", "classesDir",
		"headerClassesDir", "headerJar", "kotlinJvmTarget", "kotlinBuildFile", "emptyDir", "name")

		var kotlinKytheExtract = pctx.AndroidStaticRule("kotlinKythe",
		// TODO (b/265428637): To prevent kotlinc version skew between android builds and internal kotlin indexers (g3), consider embedding the kotlinc used by android into the kzip file.
		// This has an impact on .kzip sizes, so defer that for now.
		blueprint.RuleParams{
		Command: `rm -rf "$srcJarDir" && mkdir -p "$srcJarDir" && ` +
		`${config.ZipSyncCmd} -d $srcJarDir -l $srcJarDir/list -f ".java" -f ".kt" $srcJars && ` +
		`${config.KotlinKytheExtractor} -corpus ${kytheCorpus} --srcs @$out.rsp --srcs @"$srcJarDir/list" $commonSrcFilesList --cp @$classpath -o $out --kotlin_out $outJar ` +
		// wrap the additional kotlin args.
		// Skip Xbuild file, pass the cp explicitly.
		// Skip header jars, those should not have an effect on kythe results.
		` --args '${config.KotlincGlobalFlags} ` +
		` ${config.KotlincSuppressJDK9Warnings} ${config.JavacHeapFlags} ` +
		` $kotlincFlags -jvm-target $kotlinJvmTarget'`,
		CommandDeps: []string{
		"${config.KotlinKytheExtractor}",
		"${config.ZipSyncCmd}",
		},
		Rspfile: "$out.rsp",
		RspfileContent: "$in",
		},
		"classpath", "kotlincFlags", "commonSrcFilesList", "kotlinJvmTarget", "outJar", "srcJars", "srcJarDir",
		)

		func kotlinCommonSrcsList(ctx android.ModuleContext, commonSrcFiles android.Paths) android.OptionalPath {
		if len(commonSrcFiles) > 0 {
		// The list of common_srcs may be too long to put on the command line, but
		@@ -81,7 +104,7 @@ func kotlinCommonSrcsList(ctx android.ModuleContext, commonSrcFiles android.Path
		}

		// kotlinCompile takes .java and .kt sources and srcJars, and compiles the .kt sources into a classes jar in outputFile.
		func kotlinCompile(ctx android.ModuleContext, outputFile, headerOutputFile android.WritablePath,
		func (j *Module) kotlinCompile(ctx android.ModuleContext, outputFile, headerOutputFile android.WritablePath,
		srcFiles, commonSrcFiles, srcJars android.Paths,
		flags javaBuilderFlags) {

		@@ -127,6 +150,31 @@ func kotlinCompile(ctx android.ModuleContext, outputFile, headerOutputFile andro
		"name": kotlinName,
		},
		})

		// Emit kythe xref rule
		if (ctx.Config().EmitXrefRules()) && ctx.Module() == ctx.PrimaryModule() {
		extractionFile := outputFile.ReplaceExtension(ctx, "kzip")
		args := map[string]string{
		"classpath": classpathRspFile.String(),
		"kotlincFlags": flags.kotlincFlags,
		"kotlinJvmTarget": flags.javaVersion.StringForKotlinc(),
		"outJar": outputFile.String(),
		"srcJars": strings.Join(srcJars.Strings(), " "),
		"srcJarDir": android.PathForModuleOut(ctx, "kotlinc", "srcJars.xref").String(),
		}
		if commonSrcsList.Valid() {
		args["commonSrcFilesList"] = "--srcs @" + commonSrcsList.String()
		}
		ctx.Build(pctx, android.BuildParams{
		Rule: kotlinKytheExtract,
		Description: "kotlinKythe",
		Output: extractionFile,
		Inputs: srcFiles,
		Implicits: deps,
		Args: args,
		})
		j.kytheKotlinFiles = append(j.kytheKotlinFiles, extractionFile)
		}
		}

		var kaptStubs = pctx.AndroidRemoteStaticRule("kaptStubs", android.RemoteRuleSupports{Goma: true},