From 7fef7aaa932ad1cf1a94b1c3c8fc58b925d797ea Mon Sep 17 00:00:00 2001 From: Peter Ebden Date: Mon, 9 Sep 2024 10:35:50 +0100 Subject: [PATCH] Use more specific names for coverage variables to avoid clashes --- tools/please_go/ChangeLog | 4 ++++ tools/please_go/VERSION | 2 +- tools/please_go/cover/cover.go | 4 ++-- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/tools/please_go/ChangeLog b/tools/please_go/ChangeLog index aaf7915..ab83f74 100644 --- a/tools/please_go/ChangeLog +++ b/tools/please_go/ChangeLog @@ -1,3 +1,7 @@ +Version 1.15.1 +-------------- + * Use more specific names for coverage variables to avoid clashes + Version 1.15.0 -------------- * Add support for Go 1.23 coverage generation (#294) diff --git a/tools/please_go/VERSION b/tools/please_go/VERSION index 141f2e8..ace4423 100644 --- a/tools/please_go/VERSION +++ b/tools/please_go/VERSION @@ -1 +1 @@ -1.15.0 +1.15.1 diff --git a/tools/please_go/cover/cover.go b/tools/please_go/cover/cover.go index dd06281..156ae3c 100644 --- a/tools/please_go/cover/cover.go +++ b/tools/please_go/cover/cover.go @@ -44,9 +44,9 @@ func WriteCoverage(goTool, coverTool, covercfg, output, pkg string, srcs []strin } var cmd *exec.Cmd if coverTool != "" { - cmd = exec.Command(coverTool, append([]string{"-mode=set", "-var=goCover", "-pkgcfg", pkgConfigFile, "-outfilelist", output}, srcs...)...) + cmd = exec.Command(coverTool, append([]string{"-mode=set", "-var=_plz_goCover", "-pkgcfg", pkgConfigFile, "-outfilelist", output}, srcs...)...) } else { - cmd = exec.Command(goTool, append([]string{"tool", "cover", "-mode=set", "-var=goCover", "-pkgcfg", pkgConfigFile, "-outfilelist", output}, srcs...)...) + cmd = exec.Command(goTool, append([]string{"tool", "cover", "-mode=set", "-var=_plz_goCover", "-pkgcfg", pkgConfigFile, "-outfilelist", output}, srcs...)...) } cmd.Stdout = os.Stdout cmd.Stderr = os.Stderr