[Buildroot] [RFC v2] libvirt: new package
Carlos Santos
casantos at datacom.ind.br
Fri Sep 15 17:10:01 UTC 2017
> From: "Thomas Petazzoni" <thomas.petazzoni at free-electrons.com>
> To: "Carlos Santos" <casantos at datacom.ind.br>
> Cc: buildroot at buildroot.org
> Sent: Thursday, September 14, 2017 9:09:06 PM
> Subject: Re: [Buildroot] [RFC v2] libvirt: new package
> Hello,
>
> On Thu, 14 Sep 2017 17:36:31 -0300, Carlos Santos wrote:
>
>> +LIBVIRT_VERSION = 3.7.0
>> +LIBVIRT_SITE = http://libvirt.org/sources
>> +LIBVIRT_SOURCE = libvirt-$(LIBVIRT_VERSION).tar.xz
>> +LIBVIRT_LICENSE = LGPL-2.1+
>> +LIBVIRT_LICENSE_FILES = COPYING
>> +# LIBVIRT_INSTALL_STAGING = YES
>
> Commented line ?
A leftover from previous editions.
>> +LIBVIRT_DEPENDENCIES = libnl libtirpc libxml2 lvm2 yajl
>> +
>> +ifeq ($(BR2_PACKAGE_EUDEV),y)
>> +LIBVIRT_DEPENDENCIES += eudev libpciaccess
>> +endif
>
> Why do you handle eudev specifically here? Isn't the udev
> implementation in systemd not suitable?
Probably yes, but I didn't test with systemd, yet.
> I.e, I believe this should be:
>
> ifeq ($(BR2_PACKAGE_HAS_UDEV),y)
> LIBVIRT_DEPENDENCIES += udev libpciaccess
> endif
>
> And of course, adjust Config.in accordingly.
Done.
>> +LIBVIRT_CONF_ENV += \
>> + CFLAGS="$(TARGET_CFLAGS) `$(PKG_CONFIG_HOST_BINARY) --cflags libtirpc`" \
>> + LIBS="`$(PKG_CONFIG_HOST_BINARY) --libs libtirpc`"
>
> Why is libtirpc unconditionally necessary? The RPC implementation built
> into the C library is not sufficient ?
Fixed. I was using a uClibc-based toolchain without built-in RPC.
--
Carlos Santos (Casantos) - DATACOM, P&D
“The greatest triumph that modern PR can offer is the transcendent
success of having your words and actions judged by your reputation,
rather than the other way about.” — Christopher Hitchens
More information about the buildroot
mailing list