Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Support go1.13 -trimpath option #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aprice
Copy link

@aprice aprice commented Sep 4, 2019

Quick change to allow passing through the new -trimpath build option added in Go 1.13.

@dropwhile
Copy link

FYI - This worked great for me. Thanks @aprice.

dropwhile added a commit to cactus/go-camo that referenced this pull request Sep 8, 2019
this allows passing GOBUILD_OPTIONS=-trimpath, which isn't supported by
gox mainline yet. Support for it is in a PR[1] though.

[1]: mitchellh/gox#138
@mvdan
Copy link

mvdan commented Sep 22, 2019

Here's a temporary workaround:

export GOFLAGS="-trimpath"

gox -whatever -flags

Arguably, we could use GOFLAGS for everything, not needing to expose all flags in gox itself again.

@@ -221,6 +223,7 @@ Options:
-parallel=-1 Amount of parallelism, defaults to number of CPUs
-gocmd="go" Build command, defaults to Go
-rebuild Force rebuilding of package that were up to date
-trimpath Remove all file system paths from the resulting executable

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it some wrong tab here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants