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 7e9b1f6 commit 77e83a4
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 @@ -145,8 +145,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/[email protected]
go install github.com/hashicorp/go-changelog/cmd/changelog-build@latest
go install golang.org/x/tools/cmd/[email protected]
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 77e83a4

Please sign in to comment.