Skip to content

Commit

Permalink
deps: use install from current HEAD for hc-install (#14786)
Browse files Browse the repository at this point in the history
The `hc-install` tool we're using needed a patch for a specific bug, but that's
since been merged. We definitely want to switch to using a standard release from
that project once one is shipped with the CLI, but pinning to HEAD should keep
us for now.
  • Loading branch information
tgross committed Oct 4, 2022
1 parent 617f223 commit d063845
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/test-core.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -146,8 +146,7 @@ jobs:
run: |
make bootstrap
make generate-all
hc-install vault ${{env.VAULT_VERSION}}
hc-install consul ${{env.CONSUL_VERSION}}
sudo hc-install install -version ${{env.VAULT_VERSION}} -path /usr/local/bin vault
sudo hc-install install -version ${{env.CONSUL_VERSION}} -path /usr/local/bin consul
sudo sed -i 's!Defaults!#Defaults!g' /etc/sudoers
sudo -E env "PATH=$PATH" make test-nomad
2 changes: 1 addition & 1 deletion GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ deps: ## Install build and development dependencies
go install github.com/bufbuild/buf/cmd/buf@v0.36.0
go install github.com/hashicorp/go-changelog/cmd/changelog-build@latest
go install golang.org/x/tools/cmd/stringer@v0.1.12
go install gophers.dev/cmds/hc-install/cmd/hc-install@v1.0.2
go install github.com/hashicorp/hc-install/cmd/hc-install@4487b02cbcbb92204e3416cef9852b6ad44487b2

.PHONY: lint-deps
lint-deps: ## Install linter dependencies
Expand Down

0 comments on commit d063845

Please sign in to comment.