[libvirt] [tck PATCH 03/12] rpm: add some missing dependencies

Laine Stump laine at laine.org
Fri Jun 8 15:07:13 UTC 2018


On 06/08/2018 09:45 AM, Daniel P. Berrangé wrote:
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
>  perl-Sys-Virt-TCK.spec.PL | 15 ++++++++++++---
>  1 file changed, 12 insertions(+), 3 deletions(-)
>
> diff --git a/perl-Sys-Virt-TCK.spec.PL b/perl-Sys-Virt-TCK.spec.PL
> index 39cc72f..c92fa34 100644
> --- a/perl-Sys-Virt-TCK.spec.PL
> +++ b/perl-Sys-Virt-TCK.spec.PL
> @@ -35,8 +35,12 @@ Group: Development/Tools
>  Source: http://libvirt.org/sources/tck/%{appname}-v%{version}.tar.gz
>  Url: http://libvirt.org/
>  BuildRoot: %{_tmppath}/%{appname}-%{version}-%{release}-root-%(%{__id_u} -n)
> -Requires: perl >= %{perl_version}
> -Requires: libvirt >= 0.6.2
> +Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
> +Requires: libvirt >= 4.4.0

Are you sure it requires such a high libvirt version? This will make it
more cumbersome for downstream maintainers to test their builds of older
libvirt versions. Of course if there really is stuff that reliably fails
with older libvirt versions, then they would need to make a downstream
version of the tck that disables the failing tests anyway, but this
makes that mandatory from the get-go.

Still something I can live with though.

Reviewed-by: Laine Stump <laine at laine.org>



> +%if 0%{?fedora} || 0%{?rhel} > 7
> +BuildRequires: perl-interpreter
> +BuildRequires: perl-generators
> +%endif
>  BuildRequires: perl(accessors)
>  BuildRequires: perl(App::Prove)
>  BuildRequires: perl(Config::Record)
> @@ -57,9 +61,12 @@ BuildRequires: perl(TAP::Harness::Archive)
>  BuildRequires: perl(Test::Builder)
>  BuildRequires: perl(Test::More)
>  BuildRequires: perl(Sub::Uplevel)
> -BuildRequires: perl(Sys::Virt) >= 0.2.0
> +BuildRequires: perl(Sys::Virt) >= 0.2.1

What was this you were saying on IRC about 0.x version numbers? :-)

>  BuildRequires: perl(XML::Twig)
>  BuildRequires: perl(XML::Writer)
> +BuildRequires: perl(XML::XPath)
> +BuildRequires: perl(Test::Pod)
> +BuildRequires: perl(Test::Pod::Coverage)
>  # RPM autoprovides misses these 3
>  Requires: perl(Test::Exception)
>  Requires: perl(TAP::Formatter::HTML)
> @@ -69,6 +76,8 @@ Requires: perl(Net::OpenSSH)
>  Requires: perl(IO::Pty)
>  Requires: libguestfs-tools
>  Requires: /usr/bin/mkisofs
> +# Want to force this minimal version, so don't rely on RPM autoprov
> +Requires: perl(Sys::Virt) >= 0.2.1
>  BuildArchitectures: noarch
>  
>  %description





More information about the libvir-list mailing list