Merge "Improve searching for TestingBuildParams using the output file"

This commit is contained in:
Paul Duffin
2021-03-23 07:20:04 +00:00
committed by Gerrit Code Review

View File

@@ -585,7 +585,7 @@ func (b baseTestingComponent) maybeBuildParamsFromOutput(file string) (TestingBu
outputs = append(outputs, p.Output) outputs = append(outputs, p.Output)
} }
for _, f := range outputs { for _, f := range outputs {
if f.String() == file || f.Rel() == file { if f.String() == file || f.Rel() == file || PathRelativeToTop(f) == file {
return b.newTestingBuildParams(p), nil return b.newTestingBuildParams(p), nil
} }
searchedOutputs = append(searchedOutputs, f.Rel()) searchedOutputs = append(searchedOutputs, f.Rel())
@@ -597,8 +597,8 @@ func (b baseTestingComponent) maybeBuildParamsFromOutput(file string) (TestingBu
func (b baseTestingComponent) buildParamsFromOutput(file string) TestingBuildParams { func (b baseTestingComponent) buildParamsFromOutput(file string) TestingBuildParams {
p, searchedOutputs := b.maybeBuildParamsFromOutput(file) p, searchedOutputs := b.maybeBuildParamsFromOutput(file)
if p.Rule == nil { if p.Rule == nil {
panic(fmt.Errorf("couldn't find output %q.\nall outputs: %v", panic(fmt.Errorf("couldn't find output %q.\nall outputs:\n %s\n",
file, searchedOutputs)) file, strings.Join(searchedOutputs, "\n ")))
} }
return p return p
} }