[libvirt] [PATCH 2/3] spec: add dmidecode as prereq

[Daniel P. Berrange]

On Mon, Dec 05, 2011 at 10:38:50AM -0700, Eric Blake wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=754909 complains that
> because libvirt didn't require dmidecode, that the logs are noisy
> and virConnectGetSysinfo needlessly fails.  Even 'virt-what' requires
> dmidecode, so it's not that onerous of a dependency.  We may be
> able to drop this in the future when we move to parsing sysfs data,
> but for now, listing the dependency will help matters.
> 
> * libvirt.spec.in (Requires): Sort Requires before BuildRequires.
> Add dmidecode.
> ---
>  libvirt.spec.in |   16 +++++++++-------
>  1 files changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 462421a..35762a5 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -332,6 +332,15 @@ Requires: device-mapper
>  %if %{with_cgconfig}
>  Requires: libcgroup
>  %endif
> +# For virConnectGetSysinfo
> +Requires: dmidecode
> +# For service management
> +%if %{with_systemd}
> +Requires(post): systemd-units
> +Requires(post): systemd-sysv
> +Requires(preun): systemd-units
> +Requires(postun): systemd-units
> +%endif
> 
>  # All build-time requirements
>  BuildRequires: autoconf
> @@ -485,13 +494,6 @@ BuildRequires: nfs-utils
>  # Fedora build root suckage
>  BuildRequires: gawk
> 
> -%if %{with_systemd}
> -Requires(post): systemd-units
> -Requires(post): systemd-sysv
> -Requires(preun): systemd-units
> -Requires(postun): systemd-units
> -%endif
> -
>  %description
>  Libvirt is a C toolkit to interact with the virtualization capabilities
>  of recent versions of Linux (and other OSes). The main package includes

ACK


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|