petejkim / autoparts

package manager for nitrous.io
BSD 2-Clause "Simplified" License
81 stars 84 forks source link

Set GOROOT and GOPATH after installing the Go package. #123

Closed laktek closed 10 years ago

petejkim commented 10 years ago

This is not how you inject env vars. Please look at chruby package

petejkim commented 10 years ago

Also please set go path to be workspace/go

chuyeow commented 10 years ago

Here's a patch that uses a correct value for GOROOT: #124. You can inject the env vars on top of that patch :)

chuyeow commented 10 years ago

OK I injected the env vars in #124 as well. Please review.

I've set the GOPATH to ~/workspace/go but we might wanna use ~/workspace because the Nitrous.IO Golang boxes use ~/workspace. I understand why we don't want to just use ~/workspace though - don't wanna accidentally clobber existing files with a go get.