Re: libmemcached and include paths

Hi,

On Fri, 17 Sep 2010 16:44:58 +0900, Filip Hajny <[email protected]> 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 / [email protected]