Skip to content

Commit

Permalink
install Makefile target should depend on all, not mosquitto
Browse files Browse the repository at this point in the history
This ensures that man pages are always built, even if a user runs `make
install` without building first.

Closes #1989. Thanks to woodz-.
  • Loading branch information
ralight committed Jan 8, 2021
1 parent 7e1a818 commit 6728c05
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions ChangeLog.txt
Expand Up @@ -13,6 +13,10 @@ Broker:
before a role that was attached to the group or client is deleted.
Closes #1998.

Build:
- `install` Makefile target should depend on `all`, not `mosquitto`, to ensure
that man pages are always built. Closes #1989.

Apps:
- Disallow control characters in mosquitto_passwd usernames.
- Fix incorrect description in mosquitto_ctrl man page. Closes #1995.
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Expand Up @@ -87,7 +87,7 @@ ptest : mosquitto
utest : mosquitto
$(MAKE) -C test utest

install : mosquitto
install : all
set -e; for d in ${DIRS}; do $(MAKE) -C $${d} install; done
ifeq ($(WITH_DOCS),yes)
set -e; for d in ${DOCDIRS}; do $(MAKE) -C $${d} install; done
Expand Down

0 comments on commit 6728c05

Please sign in to comment.