Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: put GOPATH in /tmp on launchpad #20564

Merged
merged 2 commits into from
Jan 16, 2020

Conversation

fjl
Copy link
Contributor

@fjl fjl commented Jan 15, 2020

Will this change fix the Launchpad PPA build? Let's find out!

@fjl fjl force-pushed the build-lp-tmp-gopath branch from eff19c0 to f82c44d Compare January 15, 2020 14:29
@karalabe
Copy link
Member

Nope https://launchpadlibrarian.net/460839843/buildlog_ubuntu-trusty-amd64.ethereum-unstable_1.9.10+trusty_BUILDING.txt.gz

I'm also unsure you can use GOPATH if you explicitly set GO111MODULE=on forcefully, since that ignored GOPATH.

@karalabe
Copy link
Member

karalabe commented Jan 16, 2020

:P You cut off the /*

Nope, that's not it

@karalabe
Copy link
Member

Seems to work now

@karalabe karalabe added this to the 1.9.10 milestone Jan 16, 2020
@karalabe karalabe merged commit 3e97b04 into ethereum:master Jan 16, 2020
enriquefynn pushed a commit to enriquefynn/go-ethereum that referenced this pull request Mar 10, 2021
* build: put GOPATH in /tmp on launchpad

* build: don't remove GOPATH from go tool environment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants