Compare commits

...

5 Commits

Author SHA1 Message Date
Evgeny Vereshchagin d3e0662c7d Revert "ci: turn off the "upstream-systemd-ci" ppa"
This reverts commit d4ff79bbe1.
2020-07-15 08:59:25 +02:00
Filipe Brandenburger 06bb441102 test: Add "finish" rule to .PHONY 2020-07-15 08:58:46 +02:00
Zbigniew Jędrzejewski-Szmek 76830e2500
Merge pull request #16462 from keszybz/rpm-macro-warnings
Emit better errors for rpm macro misuse
2020-07-15 08:56:28 +02:00
Zbigniew Jędrzejewski-Szmek 1061fc1c17 rpm: include macro name in errors for two args macros too 2020-07-14 19:22:42 +02:00
Zbigniew Jędrzejewski-Szmek 281014b73e rpm: adjust various macros to print macro name in the error message
Based on initial patch by Jan Engelhardt <jengelh@inai.de>.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1856122.
2020-07-14 19:21:12 +02:00
3 changed files with 26 additions and 20 deletions

View File

@ -85,7 +85,7 @@ else
fi fi
# PPA with some newer build dependencies (like zstd) # PPA with some newer build dependencies (like zstd)
#add-apt-repository -y ppa:upstream-systemd-ci/systemd-ci add-apt-repository -y ppa:upstream-systemd-ci/systemd-ci
apt-get -y update apt-get -y update
apt-get -y build-dep systemd apt-get -y build-dep systemd
apt-get -y install "${PACKAGES[@]}" apt-get -y install "${PACKAGES[@]}"

View File

@ -40,11 +40,11 @@ OrderWithRequires(preun): systemd \
OrderWithRequires(postun): systemd \ OrderWithRequires(postun): systemd \
%{nil} %{nil}
%__systemd_someargs_0() %{error:This macro requires some arguments} %__systemd_someargs_0(:) %{error:The %%%1 macro requires some arguments}
%__systemd_twoargs_2() %{nil} %__systemd_twoargs_2() %{nil}
%systemd_post() \ %systemd_post() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_post}} \
if [ $1 -eq 1 ] && [ -x @bindir@/systemctl ]; then \ if [ $1 -eq 1 ] && [ -x @bindir@/systemctl ]; then \
# Initial installation \ # Initial installation \
@bindir@/systemctl --no-reload preset %{?*} || : \ @bindir@/systemctl --no-reload preset %{?*} || : \
@ -54,7 +54,7 @@ fi \
%systemd_user_post() %{expand:%systemd_post \\--global %%{?*}} %systemd_user_post() %{expand:%systemd_post \\--global %%{?*}}
%systemd_preun() \ %systemd_preun() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_preun}} \
if [ $1 -eq 0 ] && [ -x @bindir@/systemctl ]; then \ if [ $1 -eq 0 ] && [ -x @bindir@/systemctl ]; then \
# Package removal, not upgrade \ # Package removal, not upgrade \
@bindir@/systemctl --no-reload disable --now %{?*} || : \ @bindir@/systemctl --no-reload disable --now %{?*} || : \
@ -62,26 +62,32 @@ fi \
%{nil} %{nil}
%systemd_user_preun() \ %systemd_user_preun() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_user_preun}} \
if [ $1 -eq 0 ] && [ -x @bindir@/systemctl ]; then \ if [ $1 -eq 0 ] && [ -x @bindir@/systemctl ]; then \
# Package removal, not upgrade \ # Package removal, not upgrade \
@bindir@/systemctl --global disable %{?*} || : \ @bindir@/systemctl --global disable %{?*} || : \
fi \ fi \
%{nil} %{nil}
%systemd_postun() %{expand:%%{?__systemd_someargs_%#}}%{nil} %systemd_postun() \
%{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_postun}} \
%{nil}
%systemd_user_postun() %{expand:%%{?__systemd_someargs_%#}}%{nil} %systemd_user_postun() \
%{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_user_postun}} \
%{nil}
%systemd_postun_with_restart() \ %systemd_postun_with_restart() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_postun_with_restart}} \
if [ $1 -ge 1 ] && [ -x @bindir@/systemctl ]; then \ if [ $1 -ge 1 ] && [ -x @bindir@/systemctl ]; then \
# Package upgrade, not uninstall \ # Package upgrade, not uninstall \
@bindir@/systemctl try-restart %{?*} || : \ @bindir@/systemctl try-restart %{?*} || : \
fi \ fi \
%{nil} %{nil}
%systemd_user_postun_with_restart() %{expand:%%{?__systemd_someargs_%#}}%{nil} %systemd_user_postun_with_restart() \
%{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# systemd_postun_with_restart}} \
%{nil}
%udev_hwdb_update() %{nil} %udev_hwdb_update() %{nil}
@ -91,13 +97,13 @@ fi \
# Deprecated. Use %tmpfiles_create_package instead # Deprecated. Use %tmpfiles_create_package instead
%tmpfiles_create() \ %tmpfiles_create() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# tmpfiles_create}} \
[ -x @bindir@/systemd-tmpfiles ] && @bindir@/systemd-tmpfiles --create %{?*} || : \ [ -x @bindir@/systemd-tmpfiles ] && @bindir@/systemd-tmpfiles --create %{?*} || : \
%{nil} %{nil}
# Deprecated. Use %sysusers_create_package instead # Deprecated. Use %sysusers_create_package instead
%sysusers_create() \ %sysusers_create() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# sysusers_create}} \
[ -x @bindir@/systemd-sysusers ] && @bindir@/systemd-sysusers %{?*} || : \ [ -x @bindir@/systemd-sysusers ] && @bindir@/systemd-sysusers %{?*} || : \
%{nil} %{nil}
@ -121,7 +127,7 @@ SYSTEMD_INLINE_EOF\
# %files # %files
# %{_sysusersdir}/%{name}.conf # %{_sysusersdir}/%{name}.conf
%sysusers_create_package() \ %sysusers_create_package() \
%{expand:%%{?!__systemd_twoargs_%#:%%{error:This macro requires two arguments}}} \ %{expand:%%{?!__systemd_twoargs_%#:%%{error:The %%%%sysusers_create_package macro requires two arguments}}} \
systemd-sysusers --replace=%_sysusersdir/%1.conf - <<SYSTEMD_INLINE_EOF || : \ systemd-sysusers --replace=%_sysusersdir/%1.conf - <<SYSTEMD_INLINE_EOF || : \
%(cat %2) \ %(cat %2) \
SYSTEMD_INLINE_EOF\ SYSTEMD_INLINE_EOF\
@ -141,18 +147,18 @@ SYSTEMD_INLINE_EOF\
# %files # %files
# %{_tmpfilesdir}/%{name}.conf # %{_tmpfilesdir}/%{name}.conf
%tmpfiles_create_package() \ %tmpfiles_create_package() \
%{expand:%%{?!__systemd_twoargs_%#:%%{error:This macro requires two arguments}}} \ %{expand:%%{?!__systemd_twoargs_%#:%%{error:The %%%%tmpfiles_create_package macro requires two arguments}}} \
systemd-tmpfiles --replace=%_tmpfilesdir/%1.conf --create - <<SYSTEMD_INLINE_EOF || : \ systemd-tmpfiles --replace=%_tmpfilesdir/%1.conf --create - <<SYSTEMD_INLINE_EOF || : \
%(cat %2) \ %(cat %2) \
SYSTEMD_INLINE_EOF\ SYSTEMD_INLINE_EOF\
%{nil} %{nil}
%sysctl_apply() \ %sysctl_apply() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# sysctl_apply}} \
[ -x @rootlibexecdir@/systemd-sysctl ] && @rootlibexecdir@/systemd-sysctl %{?*} || : \ [ -x @rootlibexecdir@/systemd-sysctl ] && @rootlibexecdir@/systemd-sysctl %{?*} || : \
%{nil} %{nil}
%binfmt_apply() \ %binfmt_apply() \
%{expand:%%{?__systemd_someargs_%#}} \ %{expand:%%{?__systemd_someargs_%#:%%__systemd_someargs_%# binfmt_apply}} \
[ -x @rootlibexecdir@/systemd-binfmt ] && @rootlibexecdir@/systemd-binfmt %{?*} || : \ [ -x @rootlibexecdir@/systemd-binfmt ] && @rootlibexecdir@/systemd-binfmt %{?*} || : \
%{nil} %{nil}

View File

@ -13,4 +13,4 @@ all setup run clean clean-again:
finish: finish:
@basedir=../.. TEST_BASE_DIR=../ BUILD_DIR=$(BUILD_DIR) ./fini.sh --$@ @basedir=../.. TEST_BASE_DIR=../ BUILD_DIR=$(BUILD_DIR) ./fini.sh --$@
.PHONY: all setup run clean clean-again .PHONY: all setup run clean clean-again finish