Re: A basic spec file question - Mailing list pgsql-pkg-yum

From Heath Lord
Subject Re: A basic spec file question
Date
Msg-id CA+BEBhukkX5ez12RWa6b9vUHTAd7dGsz2pwt+5TWcPP-3LTGpg@mail.gmail.com
Whole thread Raw
In response to Re: A basic spec file question  (Craig Ringer <craig.ringer@enterprisedb.com>)
List pgsql-pkg-yum
Devrim,

   The issue is with the "<=" in this line:

%if 0%{?fedora} <= 31 || 0%{?rhel} <= 8 || 0%{?suse_version} >= 1315

   With the "0%{variable}" format you will get "0" whenever a variable
is not defined, so whenever you do a comparison with a "<=" it will
resolve to TRUE when that variable is undefined.  Since fedora and
rhel are never both defined at the same time the above logic will
always resolve to being TRUE.
   Anytime you use this comparison logic to determine whether the
variable is "<="  you will also have to verify that the variable is
defined as well.

Thanks,
   Heath

On Fri, Oct 2, 2020 at 5:38 AM Craig Ringer
<craig.ringer@enterprisedb.com> wrote:
>
> Add a %dump at the end of the spec file body and/or relevant parts throughout to see how it evaluated. %echo macros
areuseful too 
>
> On Fri, 2 Oct 2020, 17:08 Devrim Gündüz, <devrim@gunduz.org> wrote:
>>
>>
>>
>> Hi,
>>
>> I am probably missing a very basic thing, but... What is wrong in this
>> line? I am expecting Fedora 32 and 33 not do depend on CGAL package,
>> but then the Fedora 33 RPM depends on CGAL >= 4.7 .
>>
>>
https://git.postgresql.org/gitweb/?p=pgrpms.git;a=blob;f=rpm/redhat/master/common/sfcgal/master/sfcgal.spec;h=2678073373595d042341fe9724db6ccf1f2daacf;hb=b57f568a1eb1780a397d4c25c46c678399048e9b#l26
>>
>> What am I missing?
>>
>> Regards,
>> --
>> Devrim Gündüz
>> Open Source Solution Architect, Red Hat Certified Engineer
>> Twitter: @DevrimGunduz , @DevrimGunduzTR



pgsql-pkg-yum by date:

Previous
From: Craig Ringer
Date:
Subject: Re: A basic spec file question
Next
From: Bruno Lavoie
Date:
Subject: Adding YUM packages to repository