Skip to content
This repository was archived by the owner on Mar 27, 2024. It is now read-only.

Fix download links #32

Merged
merged 1 commit into from
Aug 23, 2017
Merged

Fix download links #32

merged 1 commit into from
Aug 23, 2017

Conversation

duggelz
Copy link
Contributor

@duggelz duggelz commented Aug 18, 2017

No description provided.

@duggelz
Copy link
Contributor Author

duggelz commented Aug 18, 2017

Also, it seems like it would be nicer to copy the executable in to $HOME/bin instead of /usr/local/bin, and get rid of the need for sudo.

nkubala
nkubala approved these changes Aug 21, 2017
View reviewed changes
Copy link
Contributor

@nkubala nkubala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix. Good point about avoiding the need for sudo, only issue with that is that $HOME/bin isn't guaranteed to be on the $PATH, so that would require an extra command. I'll add it as an option in the readme though.

@nkubala nkubala merged commit a4c2b6a into GoogleContainerTools:master Aug 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nkubala nkubala nkubala approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duggelz @nkubala