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)
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list