Re: [libvirt] [PATCH] rpm: conditionalize dep on perl for perl-interpretor split in F27
On Fri, Aug 04, 2017 at 04:37:37PM +0200, Michal Privoznik wrote:
On 08/02/2017 11:52 AM, Daniel P. Berrange wrote:
> Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com>
> ---
> libvirt.spec.in | 4 ++++
> mingw-libvirt.spec.in | 4 ++++
> 2 files changed, 8 insertions(+)
>
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index b074bd171..8abecae22 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -292,7 +292,11 @@ BuildRequires: libtool
> BuildRequires: /usr/bin/pod2man
> %endif
> BuildRequires: git
> +%if 0%{?fedora} >= 27
> +BuildRequires: perl-interpretor
s/interpretor/interpreter/
Sigh, I guess this time i *should* have cut+paste ;-)
> +%else
> BuildRequires: perl
> +%endif
> BuildRequires: python
> %if %{with_systemd}
> BuildRequires: systemd-units
> diff --git a/mingw-libvirt.spec.in b/mingw-libvirt.spec.in
> index 4efa0ddbf..553d14022 100644
> --- a/mingw-libvirt.spec.in
> +++ b/mingw-libvirt.spec.in
> @@ -59,7 +59,11 @@ BuildRequires: pkgconfig
> # Need native version for msgfmt
> BuildRequires: gettext
> BuildRequires: python
> +%if 0%{?fedora} >= 27
> +BuildRequires: perl-interpretor
again.
> +%else
> BuildRequires: perl
> +%endif
> BuildRequires: perl(Getopt::Long)
> %if 0%{?enable_autotools}
> BuildRequires: autoconf
>
ACK with that fixed.
Michal
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|