[libvirt] [PATCH] build: don't export avahi syms when unused
Eric Blake
eblake at redhat.com
Tue Jul 31 13:31:39 UTC 2012
On 07/31/2012 03:21 AM, Daniel P. Berrange wrote:
> On Mon, Jul 30, 2012 at 05:31:20PM -0600, Eric Blake wrote:
>> Detected when building --without-avahi.
>>
>> * src/libvirt_private.syms (virnetservermdns.h): Move...
>> * src/libvirt_avahi.syms: ...to new file.
>> * src/Makefile.am (USED_SYM_FILES, EXTRA_DIST): Use it.
>> ---
>>
>> I'm debating whether to push this under the build-breaker rule.
>> Unfortunately, while this solved my build on an Ubuntu machine,
>> I have other machines which are failing for other symbols (some
>> SASL related, and one for xenLinuxDomainBlockStats), so it's
>> probably better to fix everything instead of just half the problem.
>>
>> src/Makefile.am | 7 ++++++-
>> src/libvirt_avahi.syms | 13 +++++++++++++
>> src/libvirt_private.syms | 13 -------------
>> 3 files changed, 19 insertions(+), 14 deletions(-)
>> create mode 100644 src/libvirt_avahi.syms
>
> Reluctant ACK. I'm surprised I wrote it using HAVE_AVAHI.
> I would be nice to change this so that the code in the
> .c file is conditionalized, and the APIs still exist
> providing a ENOSYS error or something.
>
Sure, I can provide a v2 along these lines, then we can decide which
version we like better (the v2 will be longer, but removing the
conditionals will make maintenance of the .syms file easier).
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120731/62e34395/attachment-0001.sig>
More information about the libvir-list
mailing list