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

Re: [libvirt] [PATCH 3/4] Add a simple macro to check in which library dlopen() is, and use it.



On 02/24/2010 04:21 PM, Eric Blake wrote:
>  +  AC_SEARCH_LIBS([dlopen], [dl],
>  +    [DLOPEN_LIBS=${LIBS#${old_libs}}

The shell expansion ${a#b} is not portable to Solaris /bin/sh.  Is that a
problem for libvirt, or are we Linux-centric enough to not care?

It should definitely configure right on Solaris, however it's simpler and better to use a case statement on the cache variable instead.

Paolo


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