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

Set GOCACHE to a writable existing directory, change GOPATH #288

Merged

Conversation

metlos
Copy link
Contributor

@metlos metlos commented Jun 17, 2019

What does this PR do?

Sets GOCACHE explictly to a writable directory.

Removes the hardcoded location of /projects from the GOPATH. This is replaced by a dynamic resolution of /projects from the Che server done in eclipse-che/che-plugin-registry#165.

This is to improve experience with the go plugin. The resetting of GOCACHE makes the warning about a directory not being writable disappear. /projects should not be hardcoded because it is actually configurable in the Che server.

What issues does this PR fix or reference?

eclipse-che/che#13529
eclipse-che/che-plugin-registry#165

@benoitf
Copy link
Contributor

benoitf commented Jun 17, 2019

ci-build check is that CI is not able to build from custom user branches ?

@benoitf
Copy link
Contributor

benoitf commented Jun 17, 2019

ci-build

@metlos
Copy link
Contributor Author

metlos commented Jun 17, 2019

Please do not merge this PR until eclipse-che/che-plugin-registry#165 is merged otherwise the plugin would be broken (/projects would no longer be on GOPATH).

@monaka
Copy link
Member

monaka commented Jun 18, 2019

I merge this since eclipse-che/che-plugin-registry#165 was merged
and it's obvious this patch won't cause any build errors.
(I wish ci-build-check worked well...)

@monaka monaka merged commit d13fad0 into eclipse-che:master Jun 18, 2019
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.

3 participants