Skip to content
This repository has been archived by the owner on Feb 7, 2022. It is now read-only.

Fix $GOPATH mounting on Windows for local builds #116

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

Conversation

kevinpfab
Copy link

@kevinpfab kevinpfab commented Apr 5, 2018

Resolves #115

The EXT_GOPATH value used inside the xgo docker build script requires Linux path separators. If xgo was used on Windows and building a local build, the EXT_GOPATH argument would forward along a Windows-style path.

This patch forwards along the path with the proper Linux path separators regardless of the os.PathSeparator value.

Before: EXT_GOPATH=\ext-go\1
After: EXT_GOPATH=/ext-go/1

@luiscarrasco
Copy link

This fixes getting the errors mentioned in #115 on my windows setup.

@jicg
Copy link

jicg commented Jan 2, 2019

me too。 thanks

@thinkgos
Copy link

me too, got it ,thanks!!

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.

Regression in local paths
4 participants