[libvirt] [PATCH] Define DYNLIB_NAME on OpenBSD.

Eric Blake eblake at redhat.com
Thu Sep 6 15:19:34 UTC 2012


On 09/06/2012 01:39 AM, Daniel Veillard wrote:
> On Tue, Sep 04, 2012 at 04:56:13PM +0200, Jasper Lievisse Adriaanse wrote:
>> >From 05dd99030d865127c874d1b489b9c17412bdbb3b Mon Sep 17 00:00:00 2001
>> From: Jasper Lievisse Adriaanse <jasper at humppa.nl>
>> Date: Tue, 4 Sep 2012 16:48:51 +0200
>> Subject: [PATCH] Define DYNLIB_NAME on OpenBSD.

>> -#if defined(__linux__) || defined(__linux_gnu__) || defined(__sun__) || defined(__FreeBSD__)
>> +#if defined(__linux__) || defined(__linux_gnu__) || defined(__sun__) || defined(__FreeBSD__) || defined(__OpenBSD__)
>>  # define DYNLIB_NAME    "VBoxXPCOMC.so"
>>  #elif defined(__APPLE__)
>>  # define DYNLIB_NAME    "VBoxXPCOMC.dylib"
> 
>   This one is a no-brainer, ACK and pushed,

Agreed, although using \ line continuation for keeping within 80 columns
might have been nice.  My usual aversion to platform defines rather than
feature-based configure checks doesn't really apply to this patch,
because I don't know how to write a configure check to determine
preferred dnylib naming, short of doing platform-specific checks in the
first place.

-- 
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: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120906/adc52719/attachment-0001.sig>


More information about the libvir-list mailing list