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

[Perry N. Myers]

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

Added comment and pushed.

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