Skip to content

Commit

Permalink
rabbitmq-dist.mk: Install CLI scripts as part of the build
Browse files Browse the repository at this point in the history
... instead of the `dist` target.

[Why]
We already do that when building tests. Thus it is more consistent to do
the same.

Also, it makes sense to ensure everything is ready before the `dist`
step. For instance, an Erlang release would not depend on the `dist`
target, just the build and it would still need the CLI to be ready.
  • Loading branch information
dumbbell committed Nov 23, 2023
1 parent 7aaaf2f commit d24315d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion deps/rabbit_common/mk/rabbitmq-dist.mk
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ CLI_SCRIPTS_LOCK = $(CLI_SCRIPTS_DIR).lock
CLI_ESCRIPTS_LOCK = $(CLI_ESCRIPTS_DIR).lock

ifneq ($(filter-out rabbit_common amqp10_common rabbitmq_stream_common,$(PROJECT)),)
dist:: install-cli
app:: install-cli
test-build:: install-cli
endif

Expand Down

0 comments on commit d24315d

Please sign in to comment.