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
Add ghr package #27
Add ghr package #27
Conversation
install/Makefile
Outdated
@@ -3,6 +3,13 @@ OS ?= $(shell uname -s | tr '[:upper:]' '[:lower:']) | |||
INSTALL_PATH ?= /usr/local/bin | |||
CURL ?= curl --fail -sSL | |||
TMP ?= /tmp | |||
EXTENSION="" | |||
|
|||
ifeq ($(OS),darwin) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We cannot use a global conditional to map OS to an extension because there's no way to guarantee all projects will use the same convention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead, use this pattern:
https://github.com/cloudposse/packages/blob/master/install/Makefile#L128-L142
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use different pattern for OS distinctions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebuild readme
ghr-linux: | ||
$(CURL) -o - https://github.com/tcnksm/ghr/releases/download/v$(GHR_VERSION)/ghr_v$(GHR_VERSION)_linux_$(ARCH).tar.gz | tar --wildcards -zxO */ghr > $(INSTALL_PATH)/ghr | ||
chmod +x $(INSTALL_PATH)/ghr | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing target to intall ghr
Should be something like:
## Install ghr to easily upload multiple artifacts to GitHub Release
ghr: ghr-$(OS)
@exit 0
install/Makefile
Outdated
$(CURL) -o - https://github.com/tcnksm/ghr/releases/download/v$(GHR_VERSION)/ghr_v$(GHR_VERSION)_linux_$(ARCH).tar.gz | tar --wildcards -zxO */ghr > $(INSTALL_PATH)/ghr | ||
chmod +x $(INSTALL_PATH)/ghr | ||
|
||
ghr: ghr-$(OS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing target description.
what
ghr
why
references