From fbda1933126a36ab28b0d5ce9e1e8e7c62bd222b Mon Sep 17 00:00:00 2001
From: Marius Kittler <mkittler@suse.de>
Date: Thu, 12 Sep 2024 15:41:49 +0200
Subject: [PATCH] Exclude sub packages with many additional dependencies on SLE

As stated in https://build.suse.de/request/show/343857#comment-4023273 not
all sub packages are installable on SLE. This only concerns the `devel`
and `munin` sub packages which are really optional. So it makes sense to
avoid submitting a huge number of Haskell and Perl packages to SLE and
exclude those packages instead.

Related ticket: https://progress.opensuse.org/issues/165683
---
 dist/rpm/openQA.spec | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/dist/rpm/openQA.spec b/dist/rpm/openQA.spec
index 7e791a17b7b..eeb89e0bd56 100644
--- a/dist/rpm/openQA.spec
+++ b/dist/rpm/openQA.spec
@@ -42,6 +42,14 @@
 %else
 %bcond_without python_scripts
 %endif
+# exclude additional sub packages that would pull in a lot of extra dependencies on SLE
+%if 0%{?sle_version} && !0%{?is_opensuse}
+%bcond_with devel_package
+%bcond_with munin_package
+%else
+%bcond_without devel_package
+%bcond_without munin_package
+%endif
 # runtime requirements that also the testsuite needs
 %if %{with python_scripts}
 %define python_scripts_requires python3-base python3-requests openQA-client
@@ -155,6 +163,7 @@ revision of the operating system, reporting the errors detected for each
 combination of hardware configuration, installation options and variant of the
 operating system.
 
+%if %{with devel_package}
 %package no-selenium-devel
 Summary:        Development package pulling in all build+test dependencies except chromedriver for Selenium based tests
 Requires:       %{devel_no_selenium_requires}
@@ -168,6 +177,7 @@ Requires:       %{devel_requires}
 
 %description devel
 Development package pulling in all build+test dependencies.
+%endif
 
 %package common
 Summary:        The openQA common tools for web-frontend and workers
@@ -289,6 +299,7 @@ upgrading the system if devel:openQA packages are stable and contain updates. It
 is complementary to auto-update which also reboots the system and does updates
 regardless of whether devel:openQA contains updates.
 
+%if %{with munin_package}
 %package munin
 Summary:        Munin scripts
 Requires:       munin
@@ -299,6 +310,7 @@ Requires:       perl
 %description munin
 Use this package to install munin scripts that allow to monitor some openQA
 statistics.
+%endif
 
 
 %prep
@@ -386,10 +398,12 @@ ln -s %{_datadir}/openqa/script/openqa-label-all %{buildroot}%{_bindir}/openqa-l
 %endif
 
 # munin
+%if %{with munin_package}
 install -d -m 755 %{buildroot}/%{_prefix}/lib/munin/plugins
 install -m 755 contrib/munin/plugins/minion %{buildroot}/%{_prefix}/lib/munin/plugins/openqa_minion_
 install -d -m 755 %{buildroot}/%{_sysconfdir}/munin/plugin-conf.d
 install -m 644 contrib/munin/config/minion.config %{buildroot}/%{_sysconfdir}/munin/plugin-conf.d/openqa-minion
+%endif
 
 cd %{buildroot}
 grep -rl %{_bindir}/env . | while read file; do
@@ -649,7 +663,9 @@ fi
 %{_sysusersdir}/geekotest.conf
 %endif
 
+%if %{with devel_package}
 %files devel
+%endif
 
 %files common
 %dir %{_datadir}/openqa
@@ -783,6 +799,7 @@ fi
 %{_unitdir}/openqa-continuous-update.*
 %{_datadir}/openqa/script/openqa-continuous-update
 
+%if %{with munin_package}
 %files munin
 %defattr(-,root,root)
 %doc contrib/munin/config/minion.config
@@ -792,5 +809,6 @@ fi
 %dir %{_sysconfdir}/munin/plugin-conf.d
 %{_prefix}/lib/munin/plugins/openqa_minion_
 %config(noreplace) %{_sysconfdir}/munin/plugin-conf.d/openqa-minion
+%endif
 
 %changelog