Support API compatibility check in Soong
Test: m -j api-stubs-docs Bug: b/78245848 Change-Id: Id88a3e3716f9f07f0dc0071aa56a328c4712fcc9
This commit is contained in:
@@ -291,6 +291,7 @@ type droiddocBuilderFlags struct {
|
|||||||
metalavaStubsFlags string
|
metalavaStubsFlags string
|
||||||
metalavaAnnotationsFlags string
|
metalavaAnnotationsFlags string
|
||||||
metalavaJavadocFlags string
|
metalavaJavadocFlags string
|
||||||
|
metalavaCompatibilityCheckFlags string
|
||||||
|
|
||||||
metalavaDokkaFlags string
|
metalavaDokkaFlags string
|
||||||
}
|
}
|
||||||
@@ -1013,6 +1014,38 @@ func (d *Droiddoc) collectMetalavaDokkaFlags(ctx android.ModuleContext, implicit
|
|||||||
docStubsDir + " " + classpathArgs + " -format dac -dacRoot /reference/kotlin -output " + outDir
|
docStubsDir + " " + classpathArgs + " -format dac -dacRoot /reference/kotlin -output " + outDir
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (d *Droiddoc) collectMetalavaCompatibilityCheckFlags(ctx android.ModuleContext,
|
||||||
|
implicits *android.Paths) string {
|
||||||
|
var flags string
|
||||||
|
if d.checkCurrentApi() && !ctx.Config().IsPdkBuild() {
|
||||||
|
apiFile := ctx.ExpandSource(String(d.properties.Check_api.Current.Api_file),
|
||||||
|
"check_api.current.api_file")
|
||||||
|
removedApiFile := ctx.ExpandSource(String(d.properties.Check_api.Current.Removed_api_file),
|
||||||
|
"check_api.current_removed_api_file")
|
||||||
|
|
||||||
|
*implicits = append(*implicits, apiFile)
|
||||||
|
*implicits = append(*implicits, removedApiFile)
|
||||||
|
|
||||||
|
flags = " --check-compatibility:api:current " + apiFile.String() +
|
||||||
|
" --check-compatibility:removed:current " + removedApiFile.String() + " "
|
||||||
|
}
|
||||||
|
|
||||||
|
if d.checkLastReleasedApi() && !ctx.Config().IsPdkBuild() {
|
||||||
|
apiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Api_file),
|
||||||
|
"check_api.last_released.api_file")
|
||||||
|
removedApiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Removed_api_file),
|
||||||
|
"check_api.last_released.removed_api_file")
|
||||||
|
|
||||||
|
*implicits = append(*implicits, apiFile)
|
||||||
|
*implicits = append(*implicits, removedApiFile)
|
||||||
|
|
||||||
|
flags = flags + " --check-compatibility:api:released " + apiFile.String() +
|
||||||
|
" --check-compatibility:removed:released " + removedApiFile.String() + " "
|
||||||
|
}
|
||||||
|
|
||||||
|
return flags
|
||||||
|
}
|
||||||
|
|
||||||
func (d *Droiddoc) transformMetalava(ctx android.ModuleContext, implicits android.Paths,
|
func (d *Droiddoc) transformMetalava(ctx android.ModuleContext, implicits android.Paths,
|
||||||
implicitOutputs android.WritablePaths, outDir, docStubsDir, javaVersion,
|
implicitOutputs android.WritablePaths, outDir, docStubsDir, javaVersion,
|
||||||
bootclasspathArgs, classpathArgs, opts string) {
|
bootclasspathArgs, classpathArgs, opts string) {
|
||||||
@@ -1067,7 +1100,7 @@ func (d *Droiddoc) transformCheckApi(ctx android.ModuleContext, apiFile, removed
|
|||||||
checkApiClasspath classpath, msg, opts string, output android.WritablePath) {
|
checkApiClasspath classpath, msg, opts string, output android.WritablePath) {
|
||||||
ctx.Build(pctx, android.BuildParams{
|
ctx.Build(pctx, android.BuildParams{
|
||||||
Rule: apiCheck,
|
Rule: apiCheck,
|
||||||
Description: "Check API",
|
Description: "Doclava Check API",
|
||||||
Output: output,
|
Output: output,
|
||||||
Inputs: nil,
|
Inputs: nil,
|
||||||
Implicits: append(android.Paths{apiFile, removedApiFile, d.apiFile, d.removedApiFile},
|
Implicits: append(android.Paths{apiFile, removedApiFile, d.apiFile, d.removedApiFile},
|
||||||
@@ -1126,6 +1159,7 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
flags.doclavaStubsFlags, flags.metalavaStubsFlags = d.collectStubsFlags(ctx, &implicitOutputs)
|
flags.doclavaStubsFlags, flags.metalavaStubsFlags = d.collectStubsFlags(ctx, &implicitOutputs)
|
||||||
if Bool(d.properties.Metalava_enabled) {
|
if Bool(d.properties.Metalava_enabled) {
|
||||||
flags.metalavaAnnotationsFlags = d.collectMetalavaAnnotationsFlags(ctx, &implicits, &implicitOutputs)
|
flags.metalavaAnnotationsFlags = d.collectMetalavaAnnotationsFlags(ctx, &implicits, &implicitOutputs)
|
||||||
|
flags.metalavaCompatibilityCheckFlags = d.collectMetalavaCompatibilityCheckFlags(ctx, &implicits)
|
||||||
outDir := android.PathForModuleOut(ctx, "out").String()
|
outDir := android.PathForModuleOut(ctx, "out").String()
|
||||||
docStubsDir := android.PathForModuleOut(ctx, "docStubsDir").String()
|
docStubsDir := android.PathForModuleOut(ctx, "docStubsDir").String()
|
||||||
// TODO(nanzhang): Add a Soong property to handle documentation args.
|
// TODO(nanzhang): Add a Soong property to handle documentation args.
|
||||||
@@ -1135,7 +1169,8 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
flags.dokkaClasspathArgs, outDir, docStubsDir)
|
flags.dokkaClasspathArgs, outDir, docStubsDir)
|
||||||
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
||||||
flags.bootClasspathArgs, flags.classpathArgs, flags.metalavaStubsFlags+
|
flags.bootClasspathArgs, flags.classpathArgs, flags.metalavaStubsFlags+
|
||||||
flags.metalavaAnnotationsFlags+" "+strings.Split(flags.args, "--generate-documentation")[0]+
|
flags.metalavaAnnotationsFlags+" "+flags.metalavaCompatibilityCheckFlags+" "+
|
||||||
|
strings.Split(flags.args, "--generate-documentation")[0]+
|
||||||
flags.metalavaDokkaFlags+" "+strings.Split(flags.args, "--generate-documentation")[1])
|
flags.metalavaDokkaFlags+" "+strings.Split(flags.args, "--generate-documentation")[1])
|
||||||
} else {
|
} else {
|
||||||
flags.metalavaJavadocFlags = d.collectMetalavaJavadocFlags(
|
flags.metalavaJavadocFlags = d.collectMetalavaJavadocFlags(
|
||||||
@@ -1143,14 +1178,16 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
flags.doclavaDocsFlags = d.collectDoclavaDocsFlags(ctx, &implicits, jsilver, doclava)
|
flags.doclavaDocsFlags = d.collectDoclavaDocsFlags(ctx, &implicits, jsilver, doclava)
|
||||||
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
||||||
flags.bootClasspathArgs, flags.classpathArgs, flags.metalavaStubsFlags+
|
flags.bootClasspathArgs, flags.classpathArgs, flags.metalavaStubsFlags+
|
||||||
flags.metalavaAnnotationsFlags+" "+strings.Split(flags.args, "--generate-documentation")[0]+
|
flags.metalavaAnnotationsFlags+" "+flags.metalavaCompatibilityCheckFlags+" "+
|
||||||
|
strings.Split(flags.args, "--generate-documentation")[0]+
|
||||||
flags.metalavaJavadocFlags+flags.doclavaDocsFlags+
|
flags.metalavaJavadocFlags+flags.doclavaDocsFlags+
|
||||||
" "+strings.Split(flags.args, "--generate-documentation")[1])
|
" "+strings.Split(flags.args, "--generate-documentation")[1])
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
d.transformMetalava(ctx, implicits, implicitOutputs, outDir, docStubsDir, javaVersion,
|
||||||
flags.bootClasspathArgs, flags.classpathArgs,
|
flags.bootClasspathArgs, flags.classpathArgs,
|
||||||
flags.metalavaStubsFlags+flags.metalavaAnnotationsFlags+flags.args)
|
flags.metalavaStubsFlags+flags.metalavaAnnotationsFlags+" "+
|
||||||
|
flags.metalavaCompatibilityCheckFlags+flags.args)
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
flags.doclavaDocsFlags = d.collectDoclavaDocsFlags(ctx, &implicits, jsilver, doclava)
|
flags.doclavaDocsFlags = d.collectDoclavaDocsFlags(ctx, &implicits, jsilver, doclava)
|
||||||
@@ -1180,11 +1217,6 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
` you will need approval.\n`+
|
` you will need approval.\n`+
|
||||||
`******************************\n`, ctx.ModuleName()), String(d.properties.Check_api.Current.Args),
|
`******************************\n`, ctx.ModuleName()), String(d.properties.Check_api.Current.Args),
|
||||||
d.checkCurrentApiTimestamp)
|
d.checkCurrentApiTimestamp)
|
||||||
} else {
|
|
||||||
// TODO(nanzhang): Refactor below when Metalava support API check.
|
|
||||||
if d.apiFile == nil || d.removedApiFile == nil {
|
|
||||||
ctx.ModuleErrorf("api_filename and removed_api_filename properties cannot be empty for API check!")
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
d.updateCurrentApiTimestamp = android.PathForModuleOut(ctx, "update_current_api.timestamp")
|
d.updateCurrentApiTimestamp = android.PathForModuleOut(ctx, "update_current_api.timestamp")
|
||||||
@@ -1192,13 +1224,14 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if d.checkLastReleasedApi() && !ctx.Config().IsPdkBuild() {
|
if d.checkLastReleasedApi() && !ctx.Config().IsPdkBuild() {
|
||||||
d.checkLastReleasedApiTimestamp = android.PathForModuleOut(ctx, "check_last_released_api.timestamp")
|
|
||||||
|
|
||||||
apiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Api_file),
|
apiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Api_file),
|
||||||
"check_api.last_released.api_file")
|
"check_api.last_released.api_file")
|
||||||
removedApiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Removed_api_file),
|
removedApiFile := ctx.ExpandSource(String(d.properties.Check_api.Last_released.Removed_api_file),
|
||||||
"check_api.last_released.removed_api_file")
|
"check_api.last_released.removed_api_file")
|
||||||
|
|
||||||
|
if !Bool(d.properties.Metalava_enabled) {
|
||||||
|
d.checkLastReleasedApiTimestamp = android.PathForModuleOut(ctx, "check_last_released_api.timestamp")
|
||||||
|
|
||||||
d.transformCheckApi(ctx, apiFile, removedApiFile, checkApiClasspath,
|
d.transformCheckApi(ctx, apiFile, removedApiFile, checkApiClasspath,
|
||||||
`\n******************************\n`+
|
`\n******************************\n`+
|
||||||
`You have tried to change the API from what has been previously released in\n`+
|
`You have tried to change the API from what has been previously released in\n`+
|
||||||
@@ -1207,6 +1240,7 @@ func (d *Droiddoc) GenerateAndroidBuildActions(ctx android.ModuleContext) {
|
|||||||
d.checkLastReleasedApiTimestamp)
|
d.checkLastReleasedApiTimestamp)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
//
|
//
|
||||||
// Exported Droiddoc Directory
|
// Exported Droiddoc Directory
|
||||||
|
Reference in New Issue
Block a user