[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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

Jim Meyering wrote:
Perry Myers <pmyers 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 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

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

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

Added comment and pushed.

|=-        Red Hat, Engineering, Emerging Technologies, Boston        -=|
|=-                     Email: pmyers redhat com                      -=|
|=-         Office: +1 412 474 3552   Mobile: +1 703 362 9622         -=|
|=- GnuPG: E65E4F3D 88F9 F1C9 C2F3 1303 01FE 817C C5D2 8B91 E65E 4F3D -=|

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]