Merge "Allow users to specify extra json action data"

This commit is contained in:
Treehugger Robot 2022-05-27 13:29:54 +00:00 committed by Gerrit Code Review
commit 57d5937e6f

View file

@ -2285,6 +2285,18 @@ type JSONDataSupplier interface {
AddJSONData(d *map[string]interface{})
}
// JSONAction contains the action-related info we expose to json module graph
type JSONAction struct {
Inputs []string
Outputs []string
}
// JSONActionSupplier allows JSON representation of additional actions that are not registered in
// Ninja
type JSONActionSupplier interface {
JSONActions() []JSONAction
}
func jsonModuleFromModuleInfo(m *moduleInfo) *JsonModule {
result := &JsonModule{
jsonModuleName: *jsonModuleNameFromModuleInfo(m),
@ -2318,17 +2330,27 @@ func jsonModuleWithActionsFromModuleInfo(m *moduleInfo) *JsonModule {
Blueprint: m.relBlueprintsFile,
Module: make(map[string]interface{}),
}
var actions []map[string]interface{}
var actions []JSONAction
for _, bDef := range m.actionDefs.buildDefs {
actions = append(actions, map[string]interface{}{
"Inputs": append(
actions = append(actions, JSONAction{
Inputs: append(
getNinjaStringsWithNilPkgNames(bDef.Inputs),
getNinjaStringsWithNilPkgNames(bDef.Implicits)...),
"Outputs": append(
Outputs: append(
getNinjaStringsWithNilPkgNames(bDef.Outputs),
getNinjaStringsWithNilPkgNames(bDef.ImplicitOutputs)...),
})
}
if j, ok := m.logicModule.(JSONActionSupplier); ok {
actions = append(actions, j.JSONActions()...)
}
for _, p := range m.providers {
if j, ok := p.(JSONActionSupplier); ok {
actions = append(actions, j.JSONActions()...)
}
}
result.Module["Actions"] = actions
return result
}