~craftyguy/superd

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
1

[PATCH] makefile: add installmisc target to install everything except binaries

Details
Message ID
<20221016185119.27914-1-jami.kettunen@protonmail.com>
DKIM signature
missing
Download raw message
Patch: +7 -3
From: Jami Kettunen <jamipkettunen@gmail.com>

The "install" target still behaves like before but now one can
"make installmisc" to not install the binaries in case they're being
handled (built and installed) outside the makefile in e.g. a
distribution's package build environment.
---
 Makefile | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/Makefile b/Makefile
index 0a7b0d2..8bd6946 100644
--- a/Makefile
+++ b/Makefile
@@ -58,12 +58,14 @@ doc: $(DOCS)
clean:
	$(RM) $(DOCS) superd superctl

install: $(DOCS) superd superctl
installbins: superd superctl
	install -Dm755 superd -t $(DESTDIR)$(BINDIR)/
	install -Dm755 superctl -t $(DESTDIR)$(BINDIR)/

installmisc: $(DOCS)
	mkdir -m755 -p \
		$(DESTDIR)$(SYSCONFDIR)/superd/services \
		$(DESTDIR)$(SHAREDIR)/superd/services
	install -Dm755 superd -t $(DESTDIR)$(BINDIR)/
	install -Dm755 superctl -t $(DESTDIR)$(BINDIR)/
	install -Dm644 superd.1 -t $(DESTDIR)$(MANDIR)/man1/
	install -Dm644 superd.service.5 -t $(DESTDIR)$(MANDIR)/man5/
	install -Dm644 superctl.1 -t $(DESTDIR)$(MANDIR)/man1/
@@ -72,6 +74,8 @@ install: $(DOCS) superd superctl
	install -Dm644 completions/bash/superctl \
		$(DESTDIR)$(SHAREDIR)/bash-completion/completions/superctl

install: installbins installmisc

.PHONY: checkinstall
checkinstall:
	$(DESTDIR)$(BINDIR)/superd -v
-- 
2.38.0
Details
Message ID
<20221016225439.GF6941@craftyguy.net>
In-Reply-To
<20221016185119.27914-1-jami.kettunen@protonmail.com> (view parent)
DKIM signature
missing
Download raw message
On Sun, 16 Oct 2022 21:51:19 +0300 Jami Kettunen <jamipkettunen@gmail.com> wrote:
> From: Jami Kettunen <jamipkettunen@gmail.com>
> 
> The "install" target still behaves like before but now one can
> "make installmisc" to not install the binaries in case they're being
> handled (built and installed) outside the makefile in e.g. a
> distribution's package build environment.
> ---
>  Makefile | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 

lgtm. thanks for the patch!

-Clayton
Reply to thread Export thread (mbox)