Re: [CentOS-devel] Cockpit on CentOS 7 bugs/issues

On Thu, 25 Oct 2018 17:16:26 -0500 Johnny Hughes johnny@centos.org wrote:

OK, we figured out that part .. they used %define build_subscriptions 1 and they used to test the VALUE of that .. but now they just look at %if %{defined} .. so a 1 or a 0 will both ne true. SO .. I will have to make a change to test the value or %undefine that variable now.

Here is better idea, only define that if centos is not set.

@@ -38,7 +38,7 @@

# on RHEL 7.x we build subscriptions; superseded later by # external subscription-manager-cockpit -%if 0%{?rhel} >= 7 && 0%{?rhel} < 8 +%if 0%{?rhel} >= 7 && 0%{?rhel} < 8 && 0%{?centos} == 0 %define build_subscriptions 1 %endif