From e3f87b07bc485b2a6ff3d7b97f2c4c996836f4ec Mon Sep 17 00:00:00 2001 From: Frantisek Sumsal Date: Tue, 2 Feb 2021 17:02:12 +0100 Subject: [PATCH] ci: build with -Werror on Fedora Prompted by: https://github.com/systemd/systemd/pull/18400#issuecomment-771602705 --- .packit.yml | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/.packit.yml b/.packit.yml index 0c3c17b9da..e1ace861dd 100644 --- a/.packit.yml +++ b/.packit.yml @@ -22,8 +22,18 @@ actions: # - Patch0000-0499: backported patches from upstream # - Patch0500-9999: downstream-only patches - "sed -ri '/^Patch0[0-4][0-9]{2}+\\:.+\\.patch/d' .packit_rpm/systemd.spec" - # Build the RPMs with -Werror to catch possible compiler warnings - - "sed -i 's/^%meson /%meson --werror /' .packit_rpm/systemd.spec" + # Build the RPMs with -Werror to catch possible compiler warnings. Since + # --werror in meson doesn't seem to work with -Db_lto=true [0], let's use + # -Dc_args= and -Dcpp_args= instead. + # + # Exceptions: + # - use -Wno-deprecated-declarations to get around mallinfo() use in + # basic/selinux-util.c + # - don't use -Werror on x86 architectures, otherwise all function checks + # will fail (with error: cast from pointer to integer of different size) + # + # [0] https://github.com/mesonbuild/meson/issues/7360 + - 'sed -i "/^CONFIGURE_OPTS=(/a%ifnarch i386 i686\n-Dc_args=\"-Werror -Wno-deprecated-declarations\" -Dcpp_args=\"-Werror -Wno-deprecated-declarations\"\n%endif" .packit_rpm/systemd.spec' jobs: - job: copr_build