]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
ci: build with -Werror on Fedora
authorFrantisek Sumsal <frantisek@sumsal.cz>
Tue, 2 Feb 2021 16:02:12 +0000 (17:02 +0100)
committerLuca Boccassi <luca.boccassi@gmail.com>
Tue, 2 Feb 2021 23:17:15 +0000 (23:17 +0000)
Prompted by: https://github.com/systemd/systemd/pull/18400#issuecomment-771602705

.packit.yml

index 0c3c17b9da96e25a0c7f0e05b41a2321d3ef5ff2..e1ace861ddc4cdd78c7c22a3f88ce3e568f9a331 100644 (file)
@@ -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