Merge "HostToolPath should return install path" am: 65c74d9f89

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1396128

Change-Id: I628ebb49b3315bed63c3c6d9174d9b96ccd38ba9
This commit is contained in:
Treehugger Robot 2020-08-12 03:03:28 +00:00 committed by Automerger Merge Worker
commit a2e8000dce
2 changed files with 7 additions and 2 deletions

View file

@ -36,6 +36,11 @@ func TestPreferDynamicBinary(t *testing.T) {
fizzBuzz := ctx.ModuleForTests("fizz-buzz", "linux_glibc_x86_64").Output("fizz-buzz")
fizzBuzzDynamic := ctx.ModuleForTests("fizz-buzz-dynamic", "linux_glibc_x86_64").Output("fizz-buzz-dynamic")
path := ctx.ModuleForTests("fizz-buzz", "linux_glibc_x86_64").Module().(*Module).HostToolPath()
if g, w := path.String(), "/host/linux-x86/bin/fizz-buzz"; !strings.Contains(g, w) {
t.Errorf("wrong host tool path, expected %q got %q", w, g)
}
// Do not compile binary modules with the --test flag.
flags := fizzBuzzDynamic.Args["rustcFlags"]
if strings.Contains(flags, "--test") {

View file

@ -1057,8 +1057,8 @@ func (mod *Module) HostToolPath() android.OptionalPath {
if !mod.Host() {
return android.OptionalPath{}
}
if _, ok := mod.compiler.(*binaryDecorator); ok {
return mod.outputFile
if binary, ok := mod.compiler.(*binaryDecorator); ok {
return android.OptionalPathForPath(binary.baseCompiler.path)
}
return android.OptionalPath{}
}