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

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
 
 %define app_root %{_datadir}/%{name}
 
-- 
1.5.5.1

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