Merge pull request #492 from goreleaser/gnutar

fix: use gnu-tar on macOS

pipeline/fpm/fpm.go

@@ -4,8 +4,10 @@ package fpm
 import (
 	"fmt"
 	"io/ioutil"
+	"os"
 	"os/exec"
 	"path/filepath"
+	"strings"
 
 	"github.com/apex/log"
 	"github.com/pkg/errors"
@@ -21,6 +23,9 @@ import (
 // ErrNoFPM is shown when fpm cannot be found in $PATH
 var ErrNoFPM = errors.New("fpm not present in $PATH")
 
+// path to gnu-tar on macOS when installed with homebrew
+const gnuTarPath = "/usr/local/opt/gnu-tar/libexec/gnubin"
+
 // Pipe for fpm packaging
 type Pipe struct{}
 
@@ -114,8 +119,7 @@ func create(ctx *context.Context, format, arch string, binaries []artifact.Artif
 	}
 
 	log.WithField("args", options).Debug("creating fpm package")
-	/* #nosec */
+	if out, err := cmd(options).CombinedOutput(); err != nil {
-	if out, err := exec.Command("fpm", options...).CombinedOutput(); err != nil {
 		return errors.Wrap(err, string(out))
 	}
 	ctx.Artifacts.Add(artifact.Artifact{
@@ -129,6 +133,19 @@ func create(ctx *context.Context, format, arch string, binaries []artifact.Artif
 	return nil
 }
 
+func cmd(options []string) *exec.Cmd {
+	/* #nosec */
+	var cmd = exec.Command("fpm", options...)
+	cmd.Env = []string{fmt.Sprintf("PATH=%s:%s", gnuTarPath, os.Getenv("PATH"))}
+	for _, env := range os.Environ() {
+		if strings.HasPrefix(env, "PATH=") {
+			continue
+		}
+		cmd.Env = append(cmd.Env, env)
+	}
+	return cmd
+}
+
 func basicOptions(ctx *context.Context, workdir, format, arch, file string) []string {
 	var options = []string{
 		"--input-type", "dir",

pipeline/fpm/fpm_test.go

@@ -122,6 +122,12 @@ func TestCreateFileDoesntExist(t *testing.T) {
 	assert.Error(t, Pipe{}.Run(ctx))
 }
 
+func TestCmd(t *testing.T) {
+	cmd := cmd([]string{"--help"})
+	assert.NotEmpty(t, cmd.Env)
+	assert.Contains(t, cmd.Env[0], gnuTarPath)
+}
+
 func TestDefault(t *testing.T) {
 	var ctx = &context.Context{
 		Config: config.Project{