Skip to content

Commit

Permalink
fix(makefile): remove remove target dependency to dependencies ta…
Browse files Browse the repository at this point in the history
…rget (#10494)

To preserve the previous behaviour.
  • Loading branch information
fffonion authored Mar 16, 2023
1 parent ecbf062 commit 66895f6
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -147,10 +147,10 @@ fix-windows:
# the following targets are kept for backwards compatibility
# dev is renamed to dev-legacy
remove:
$(warning 'remove' target is deprecated, please use `dev` instead)
$(warning 'remove' target is deprecated, please use `make dev` instead)
-@luarocks remove kong

dependencies: bin/grpcurl remove
dependencies: bin/grpcurl
$(warning 'dependencies' target is deprecated, this is now not needed when using `make dev`, but are kept for installation that are not built by Bazel)

for rock in $(DEV_ROCKS) ; do \
Expand All @@ -165,4 +165,4 @@ dependencies: bin/grpcurl remove
install-legacy:
@luarocks make OPENSSL_DIR=$(OPENSSL_DIR) CRYPTO_DIR=$(OPENSSL_DIR) YAML_DIR=$(YAML_DIR)

dev-legacy: bin/grpcurl remove install-legacy dependencies
dev-legacy: remove install-legacy dependencies

1 comment on commit 66895f6

@khcp-gha-bot
Copy link

Choose a reason for hiding this comment

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

Bazel Build

Docker image available kong/kong:66895f68f1d340e3e689780bd96f821c55548acf
Artifacts available https://github.com/Kong/kong/actions/runs/4434481300

Please sign in to comment.