Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Use GOPATH/pkg/depcache instead of GOPATH/depcache #79

Merged
merged 1 commit into from
Jan 5, 2017
Merged

Use GOPATH/pkg/depcache instead of GOPATH/depcache #79

merged 1 commit into from
Jan 5, 2017

Conversation

sdboyer
Copy link
Member

@sdboyer sdboyer commented Dec 31, 2016

Per discussion in #77

This is a potentially bikesheddy (golang/go#17262, golang/go#4719, etc.), so let's get some 👍 before going ahead

@jessfraz
Copy link
Contributor

jessfraz commented Jan 4, 2017

ooo interesting I didn't know about golang/go#4719, LGTM but im sure adg knows more

@freeformz
Copy link

$GOPATH/pkg/depcache seems strange to me, but not too strange.

@jessfraz
Copy link
Contributor

jessfraz commented Jan 5, 2017

yeah it kinda seems like ls-ing in the dir primarily with folders named after archs is odd but idk

@sdboyer
Copy link
Member Author

sdboyer commented Jan 5, 2017

i tend to agree that it's a bit odd semantically, but i'm fine with doing it for now b/c it's an easy thing for us to change later, and it solves the $HOME/depcache problem that we currently have

@freeformz freeformz merged commit 8e76105 into golang:master Jan 5, 2017
@jessfraz
Copy link
Contributor

jessfraz commented Jan 5, 2017

ooo makes sense

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