Fix bug: allow multiple gen_notice modules.
Test: m cts dist reportmissinglicenses Change-Id: I07963c83eaddfe363a04871e813b56fe7f1465ad
This commit is contained in:
@@ -47,14 +47,14 @@ func modulesLicenseMetadata(ctx BuilderContext, modules ...Module) Paths {
|
||||
}
|
||||
|
||||
// buildNoticeOutputFromLicenseMetadata writes out a notice file.
|
||||
func buildNoticeOutputFromLicenseMetadata(ctx BuilderContext, tool, name string, outputFile WritablePath, libraryName, stripPrefix string, modules ...Module) {
|
||||
func buildNoticeOutputFromLicenseMetadata(ctx BuilderContext, tool, ruleName string, outputFile WritablePath, libraryName, stripPrefix string, modules ...Module) {
|
||||
depsFile := outputFile.ReplaceExtension(ctx, strings.TrimPrefix(outputFile.Ext()+".d", "."))
|
||||
rule := NewRuleBuilder(pctx, ctx)
|
||||
if len(modules) == 0 {
|
||||
if mctx, ok := ctx.(ModuleContext); ok {
|
||||
modules = []Module{mctx.Module()}
|
||||
} else {
|
||||
panic(fmt.Errorf("%s %q needs a module to generate the notice for", name, libraryName))
|
||||
panic(fmt.Errorf("%s %q needs a module to generate the notice for", ruleName, libraryName))
|
||||
}
|
||||
}
|
||||
if libraryName == "" {
|
||||
@@ -75,26 +75,26 @@ func buildNoticeOutputFromLicenseMetadata(ctx BuilderContext, tool, name string,
|
||||
cmd = cmd.FlagWithArg("--product ", libraryName)
|
||||
}
|
||||
cmd = cmd.Inputs(modulesLicenseMetadata(ctx, modules...))
|
||||
rule.Build(name, "container notice file")
|
||||
rule.Build(ruleName, "container notice file")
|
||||
}
|
||||
|
||||
// BuildNoticeTextOutputFromLicenseMetadata writes out a notice text file based
|
||||
// on the license metadata files for the input `modules` defaulting to the
|
||||
// current context module if none given.
|
||||
func BuildNoticeTextOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "textnotice", "text_notice", outputFile, libraryName, stripPrefix, modules...)
|
||||
func BuildNoticeTextOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, ruleName, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "textnotice", "text_notice_"+ruleName, outputFile, libraryName, stripPrefix, modules...)
|
||||
}
|
||||
|
||||
// BuildNoticeHtmlOutputFromLicenseMetadata writes out a notice text file based
|
||||
// on the license metadata files for the input `modules` defaulting to the
|
||||
// current context module if none given.
|
||||
func BuildNoticeHtmlOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "htmlnotice", "html_notice", outputFile, libraryName, stripPrefix, modules...)
|
||||
func BuildNoticeHtmlOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, ruleName, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "htmlnotice", "html_notice_"+ruleName, outputFile, libraryName, stripPrefix, modules...)
|
||||
}
|
||||
|
||||
// BuildNoticeXmlOutputFromLicenseMetadata writes out a notice text file based
|
||||
// on the license metadata files for the input `modules` defaulting to the
|
||||
// current context module if none given.
|
||||
func BuildNoticeXmlOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "xmlnotice", "xml_notice", outputFile, libraryName, stripPrefix, modules...)
|
||||
func BuildNoticeXmlOutputFromLicenseMetadata(ctx BuilderContext, outputFile WritablePath, ruleName, libraryName, stripPrefix string, modules ...Module) {
|
||||
buildNoticeOutputFromLicenseMetadata(ctx, "xmlnotice", "xml_notice_"+ruleName, outputFile, libraryName, stripPrefix, modules...)
|
||||
}
|
||||
|
Reference in New Issue
Block a user