summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2020-11-26 18:15:42 +0100
committerThomas Haller <thaller@redhat.com>2020-11-26 18:31:46 +0100
commite68e5c0a4c36ab6fe7cf4793f77ca741179690ce (patch)
tree8815d88c4e83bc5be91ac11afd803efbe099d46d
parente2ed707c877c5931446626b34475d1073382d80e (diff)
downloadNetworkManager-e68e5c0a4c36ab6fe7cf4793f77ca741179690ce.tar.gz
contrib/rpm: on rhel-9 let a test failure or compiler warning fail the build
We always run the unit tests during package build and also enable all compiler warnings. However, by default we used to ignore failures. That is, because rebuilding a package on another, future distro led to frequent, annoying build failures. Especially compiler warnings appear easily when using a different compiler version. The default mostly matters here when you want to build the package in brew/koji, where you don't have a possibility to explicitly select the build option. Note that rpmdiff detects failures in the build log, and thus we usually would not miss failures for builds we add to errata. Also, all our CI tests build packages with a manner where they would not allow a failure of the unit tests. So, we run these unit tests frequently and in a manner where we notice a failure. For rhel-9 builds, change the default here and let test failures and compiler warnings be fatal to the build.
-rw-r--r--contrib/fedora/rpm/NetworkManager.spec4
1 files changed, 4 insertions, 0 deletions
diff --git a/contrib/fedora/rpm/NetworkManager.spec b/contrib/fedora/rpm/NetworkManager.spec
index 343e291998..8c0caebffa 100644
--- a/contrib/fedora/rpm/NetworkManager.spec
+++ b/contrib/fedora/rpm/NetworkManager.spec
@@ -58,7 +58,11 @@
%bcond_without nm_cloud_setup
%bcond_without regen_docs
%bcond_with debug
+%if 0%{?rhel} >= 9
+%bcond_without test
+%else
%bcond_with test
+%endif
%if 0%{?fedora} >= 33 || 0%{?rhel} >= 9
%bcond_without lto
%else