23 Jun
2020
23 Jun
'20
8:13 p.m.
Hi Daniel
Think this change
```
%if 0%{?rhel} || 0%{?centos_ver}
%if 0%{?rhel} > 6 || 0%{?centos_ver} > 6
BuildRequires: pkgconfig
%endif
%else
BuildRequires: pkgconfig
%endif
```
Is equal to
```
%if 0%{?rhel}
%if 0%{?rhel} > 6
BuildRequires: pkgconfig
%endif
%else
BuildRequires: pkgconfig
%endif
```
And equal to
```
%if 0%{?rhel} > 6
BuildRequires: pkgconfig
%else
BuildRequires: pkgconfig
%endif
```
As `pkgconfig` is present in all RPM supported dists think we can use
```
BuildRequires: pkgconfig
```
What you think?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2368#issuecomment-648392981