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

Re: [libvirt] [RFC] [PATCH 1/3 v3] vepa+vsi: Introduce dependency on libnl



On 05/25/2010 02:34 PM, Stefan Berger wrote:
On Tue, 2010-05-25 at 11:58 -0600, Eric Blake wrote:
On 05/24/2010 06:24 AM, Stefan Berger wrote:
@@ -283,6 +289,10 @@ BuildRequires: yajl-devel
  %if %{with_libpcap}
  BuildRequires: libpcap-devel
  %endif
+%if %{with_libnl}
+BuildRequires: libnl-devel
+Requires: libnl
+%endif
Correction - it was pointed out to me on IRC that the 'Requires: libnl'
line is not necessary (compare with the libpcap segment above) - the rpm
build process will automatically picks up any Requires dependencies
discovered during .so tracing.  If you haven't pushed yet, then adjust
that one line.
I haven't pushed it but will remove the line. Still waiting for an ACK
on the patch.

    Stefan

I considered the ACK in Eric's previous message as binding. If one isn't enough, then I'll ACK as well, since I've been using it and it's doing the right thing.

(BTW, in case you didn't see my message on IRC, the other patches are being reviewed right now, so should be ready for push RSN)


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