make: Fix CONFIG_SUBDIR
All checks were successful
CI / Packaging - Kali Linux (pull_request) Successful in 3m50s
CI / Packaging - OpenSUSE Tumbleweed (pull_request) Successful in 4m17s
CI / Packaging test (pull_request) Successful in 0s
CI / Packaging - Kali Linux (push) Successful in 3m49s
CI / Packaging - OpenSUSE Tumbleweed (push) Successful in 4m7s
CI / Packaging test (push) Successful in 0s
All checks were successful
CI / Packaging - Kali Linux (pull_request) Successful in 3m50s
CI / Packaging - OpenSUSE Tumbleweed (pull_request) Successful in 4m17s
CI / Packaging test (pull_request) Successful in 0s
CI / Packaging - Kali Linux (push) Successful in 3m49s
CI / Packaging - OpenSUSE Tumbleweed (push) Successful in 4m7s
CI / Packaging test (push) Successful in 0s
The existence of CONFIG_SUBDIR hijacks INSTALL_CFGDIR to a subdirectory (by default /etc/opt/<package>/$(CONFIG_SUBDIR)). Then std_install_rules apply, and (the bent) INSTALL_CFGDIR is installed, but its parent directory, the original $(INSTALL_CFGDIR), is not a prerequisite of install anylonger, and has no rule anymore, hence log-install is never run for it. Instead it's implicitly created by log-install -D $(INSTALL_CFGDIR). This commit gives /etc/opt/<package> std-install rule and variables back as CFGTOPDIR variants, and inserts it early into the install target's prerequisite list. Signed-off-by: Jan Lindemann <jan@janware.com>
This commit is contained in:
parent
ddf310d3ad
commit
f33f47c2b3
4 changed files with 22 additions and 4 deletions
19
make/conf.mk
19
make/conf.mk
|
|
@ -4,15 +4,26 @@ include $(JWBDIR)/make/dev-utils.mk
|
||||||
|
|
||||||
all:
|
all:
|
||||||
clean: clean.conf
|
clean: clean.conf
|
||||||
install: install_CFG install_files_SYSCFG install_files_LOGROT install_files_CRONTAB install_files_RSYSLOG \
|
install: \
|
||||||
install_files_APACHE_CONF install_files_SYSTEMD install_files_INIT install_files_TMPFILES $(CONF_D)
|
$(INSTALL_CFGTOPDIR) \
|
||||||
|
install_CFG \
|
||||||
|
install_files_SYSCFG \
|
||||||
|
install_files_LOGROT \
|
||||||
|
install_files_CRONTAB \
|
||||||
|
install_files_RSYSLOG \
|
||||||
|
install_files_APACHE_CONF \
|
||||||
|
install_files_SYSTEMD \
|
||||||
|
install_files_INIT \
|
||||||
|
install_files_TMPFILES \
|
||||||
|
$(CONF_D)
|
||||||
test: all
|
test: all
|
||||||
|
|
||||||
clean.conf:
|
clean.conf:
|
||||||
$(RM) -rf *~ .*.swp *.done
|
$(RM) -rf *~ .*.swp *.done
|
||||||
|
|
||||||
ifneq ($(CONFIG_SUBDIR),)
|
ifneq ($(INSTALL_CFGTOPDIR),)
|
||||||
$(INSTALLED_CFG): $(INSTALL_CFGDIR)
|
$(INSTALL_CFGTOPDIR):
|
||||||
|
$(INSTALL) -D -d -m $(CFGTOPDIRMODE) -o $(CFGTOPDIROWNER) -g $(CFGTOPDIRGROUP) $@
|
||||||
endif
|
endif
|
||||||
|
|
||||||
ifneq ($(CONF_D),)
|
ifneq ($(CONF_D),)
|
||||||
|
|
|
||||||
|
|
@ -72,6 +72,9 @@ CFGMODE ?= 660
|
||||||
CFGDIROWNER ?= $(EXEDIROWNER)
|
CFGDIROWNER ?= $(EXEDIROWNER)
|
||||||
CFGDIRGROUP ?= $(EXEDIRGROUP)
|
CFGDIRGROUP ?= $(EXEDIRGROUP)
|
||||||
CFGDIRMODE ?= $(EXEDIRMODE)
|
CFGDIRMODE ?= $(EXEDIRMODE)
|
||||||
|
CFGTOPDIROWNER ?= $(CFGDIROWNER)
|
||||||
|
CFGTOPDIRGROUP ?= $(CFGDIRGROUP)
|
||||||
|
CFGTOPDIRMODE ?= $(CFGDIRMODE)
|
||||||
SYSCFGOWNER ?= $(CFGOWNER)
|
SYSCFGOWNER ?= $(CFGOWNER)
|
||||||
SYSCFGGROUP ?= $(CFGGROUP)
|
SYSCFGGROUP ?= $(CFGGROUP)
|
||||||
SYSCFGMODE ?= $(CFGMODE)
|
SYSCFGMODE ?= $(CFGMODE)
|
||||||
|
|
|
||||||
|
|
@ -72,6 +72,9 @@ CFGMODE ?= 664
|
||||||
CFGDIROWNER ?= $(EXEDIROWNER)
|
CFGDIROWNER ?= $(EXEDIROWNER)
|
||||||
CFGDIRGROUP ?= $(EXEDIRGROUP)
|
CFGDIRGROUP ?= $(EXEDIRGROUP)
|
||||||
CFGDIRMODE ?= $(EXEDIRMODE)
|
CFGDIRMODE ?= $(EXEDIRMODE)
|
||||||
|
CFGTOPDIROWNER ?= $(CFGDIROWNER)
|
||||||
|
CFGTOPDIRGROUP ?= $(CFGDIRGROUP)
|
||||||
|
CFGTOPDIRMODE ?= $(CFGDIRMODE)
|
||||||
SYSCFGOWNER ?= $(CFGOWNER)
|
SYSCFGOWNER ?= $(CFGOWNER)
|
||||||
SYSCFGGROUP ?= $(CFGGROUP)
|
SYSCFGGROUP ?= $(CFGGROUP)
|
||||||
SYSCFGMODE ?= $(CFGMODE)
|
SYSCFGMODE ?= $(CFGMODE)
|
||||||
|
|
|
||||||
|
|
@ -316,6 +316,7 @@ BUILD_CFGDIR = $(TOPDIR)/conf
|
||||||
ifeq ($(CONFIG_SUBDIR),)
|
ifeq ($(CONFIG_SUBDIR),)
|
||||||
INSTALL_CFGDIR ?= $(CONFIG_DIR)
|
INSTALL_CFGDIR ?= $(CONFIG_DIR)
|
||||||
else
|
else
|
||||||
|
INSTALL_CFGTOPDIR ?= $(CONFIG_DIR)
|
||||||
INSTALL_CFGDIR ?= $(CONFIG_DIR)/$(CONFIG_SUBDIR)
|
INSTALL_CFGDIR ?= $(CONFIG_DIR)/$(CONFIG_SUBDIR)
|
||||||
endif
|
endif
|
||||||
LOCAL_CFG ?= $(filter-out %.site.conf, $(filter-out rsyslog-%, $(wildcard *.conf *.xml *.bprof *.jw-tmpl *.ini *.yaml *.toml))) $(LOCAL_EXTRA_CFG)
|
LOCAL_CFG ?= $(filter-out %.site.conf, $(filter-out rsyslog-%, $(wildcard *.conf *.xml *.bprof *.jw-tmpl *.ini *.yaml *.toml))) $(LOCAL_EXTRA_CFG)
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue