On Thu, 25 Oct 2018 17:16:26 -0500 Johnny Hughes <johnny at 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 -- Tuomo Soini <tis at foobar.fi> Foobar Linux services +358 40 5240030 Foobar Oy <https://foobar.fi/> -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: OpenPGP digital signature URL: <http://lists.centos.org/pipermail/centos-devel/attachments/20181026/fda5643a/attachment-0008.sig>