go/goreleaser
go/goreleaser
1
0
Fork 0
mirror of https://github.com/goreleaser/goreleaser.git synced 2025-03-19 20:57:53 +02:00
Code Issues Releases Activity

refactor(build): remove unused param (#3248)

Browse Source 
we dont need to check the go version anymore, so we can remove all this
now.

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
This commit is contained in:
Carlos Alexandro Becker 2022-07-22 09:51:39 -03:00 committed by GitHub
parent 7aeebb12f2
commit 236e987ced
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 59 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
internal/builders/buildtarget/targets.go
View File

@ -4,8 +4,6 @@ package buildtarget
import (
"fmt"
"os"
"os/exec"
"strings"
"github.com/caarlos0/log"
@ -25,14 +23,6 @@ func (t target) String() string {
// List compiles the list of targets for the given builds.
func List(build config.Build) ([]string, error) {
version, err := goVersion(build)
if err != nil {
return nil, err
}
return matrix(build, version)
}
func matrix(build config.Build, version []byte) ([]string, error) {
// nolint:prealloc
var targets []target
// nolint:prealloc
@ -135,23 +125,6 @@ func ignored(build config.Build, target target) bool {
return false
}
func goVersion(build config.Build) ([]byte, error) {
cmd := exec.Command(build.GoBinary, "version")
// If the build.Dir is acessible, set the cmd dir to it in case
// of reletive path to GoBinary
if fileInfo, err := os.Stat(build.Dir); err == nil {
if !fileInfo.IsDir() {
return nil, fmt.Errorf("invalid builds.dir property, it should be a directory: %s", build.Dir)
}
cmd.Dir = build.Dir
}
bts, err := cmd.CombinedOutput()
if err != nil {
return nil, fmt.Errorf("unable to determine version of go binary (%s): %w", build.GoBinary, err)
}
return bts, nil
}
func valid(target target) bool {
return contains(target.os+target.arch, validTargets)
}

43
internal/builders/buildtarget/targets_test.go
View File

@ -67,7 +67,7 @@ func TestAllBuildTargets(t *testing.T) {
}
t.Run("go 1.18", func(t *testing.T) {
result, err := matrix(build, []byte("go version go1.18.0"))
result, err := List(build)
require.NoError(t, err)
require.Equal(t, []string{
"linux_386",
@ -111,46 +111,46 @@ func TestAllBuildTargets(t *testing.T) {
})
t.Run("invalid goos", func(t *testing.T) {
_, err := matrix(config.Build{
_, err := List(config.Build{
Goos: []string{"invalid"},
Goarch: []string{"amd64"},
Goamd64: []string{"v2"},
}, []byte("go version go1.18.0"))
})
require.EqualError(t, err, "invalid goos: invalid")
})
t.Run("invalid goarch", func(t *testing.T) {
_, err := matrix(config.Build{
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"invalid"},
}, []byte("go version go1.18.0"))
})
require.EqualError(t, err, "invalid goarch: invalid")
})
t.Run("invalid goarm", func(t *testing.T) {
_, err := matrix(config.Build{
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"arm"},
Goarm: []string{"invalid"},
}, []byte("go version go1.18.0"))
})
require.EqualError(t, err, "invalid goarm: invalid")
})
t.Run("invalid gomips", func(t *testing.T) {
_, err := matrix(config.Build{
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"mips"},
Gomips: []string{"invalid"},
}, []byte("go version go1.18.0"))
})
require.EqualError(t, err, "invalid gomips: invalid")
})
t.Run("invalid goamd64", func(t *testing.T) {
_, err := matrix(config.Build{
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"amd64"},
Goamd64: []string{"invalid"},
}, []byte("go version go1.18.0"))
})
require.EqualError(t, err, "invalid goamd64: invalid")
})
}
@ -236,25 +236,4 @@ func TestList(t *testing.T) {
require.NoError(t, err)
require.Equal(t, []string{"linux_amd64_v2"}, targets)
})
t.Run("error with dir", func(t *testing.T) {
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"amd64"},
Goamd64: []string{"v2"},
GoBinary: "go",
Dir: "targets.go",
})
require.EqualError(t, err, "invalid builds.dir property, it should be a directory: targets.go")
})
t.Run("fail", func(t *testing.T) {
_, err := List(config.Build{
Goos: []string{"linux"},
Goarch: []string{"amd64"},
Goamd64: []string{"v2"},
GoBinary: "nope",
})
require.EqualError(t, err, `unable to determine version of go binary (nope): exec: "nope": executable file not found in $PATH`)
})
}