[Ovirt-devel] [PATCH appliance] Added in libvirt, chkconfig and kvm as Requires for the appliance

[Jim Meyering]

Perry Myers <pmyers at redhat.com> wrote:

> The create-ovirt-appliance script requires libvirt and chkconfig
> to set up the appliance and network configurations.  brctl is
> brought in as a dependency of libvirt
>
> And the appliance requires libvirt and kvm to actually run.
>
> Any version of chkconfig should work, but as of right now we
> require specific versions of libvirt and kvm from the ovirt
> repositories.  So the version checks for those packages
> are somewhat more specific.  This will change as we get the
> migration patches out of the ovirt repo and upstream.
>
> Signed-off-by: Perry Myers <pmyers at redhat.com>
> ---
>  ovirt-appliance.spec |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/ovirt-appliance.spec b/ovirt-appliance.spec
> index 9cb1984..f06b65e 100644
> --- a/ovirt-appliance.spec
> +++ b/ovirt-appliance.spec
> @@ -20,6 +20,9 @@ BuildRequires: livecd-tools >= 017
>  BuildRequires: syslinux
>  BuildRequires: qemu-img
>  BuildRequires: wget
> +Requires: chkconfig
> +Requires: libvirt >= 0.4.4-2ovirt2
> +Requires: kvm >= 72-3ovirt3

ACK.
it might be nice to add a comment reminding us to make the changes:

# FIXME: remove ovirt* suffix when the required functionality is upstream