Re: libmemcached and include paths

["OBATA Akio" <obache%users.sourceforge.net@localhost>]

Hi,

On Fri, 17 Sep 2010 16:44:58 +0900, Filip Hajny <filip%joyent.com@localhost> 
wrote:

> Hi guys,
>
> I'm attempting to bump wip/php-memcached (and subsequently wip/libmemcached 
> too) to more recent versions, and hit a problem with include paths as defined 
> by libmemcached/buildlink3.mk. The problem is that php_memcached.c triggers 
> (through PHP's php_config.h) the inclusion of the system string.h header file, 
> however that conflicts with ${PREFIX}/include/libmemcached/string.h.
>
> Obviously I could just change the libmemcached/buildlink3.mk file to not add 
> the ${PREFIX}/include/libmemcached include path to the chain, but that may 
> break something elsewhere. I was wondering, is there a common way out of this, 
> or is it just an unfortunate file naming on the part of libmemcached?

I feel that header files are renamed.
i.e. include/libmemcached/memcached_string.h => include/libmemcached/string.h
 
<http://bazaar.launchpad.net/~libmemcached-developers/libmemcached/trunk/revision/634>

So you can drop BUILDLINK_INCDIRS.libmemcached from buildlink3.mk.
If it break something, it should be fixed to catch up the change.
Probably, following:
< #include "memcached_string.h"
> #include "libmemcached/string.h"

--
OBATA Akio / obache%users.sourceforge.net@localhost