Am 15.05.2011 um 21:53 schrieb Peter Kümmel: > On 15.05.2011 20:47, Kornel Benko wrote: >> Am Samstag, 14. Mai 2011 schrieb kuem...@lyx.org: >>> Author: kuemmel >>> Date: Sat May 14 14:49:26 2011 >>> New Revision: 38754 >>> URL: http://www.lyx.org/trac/changeset/38754 >>> >>> Log: >>> cmake: remove mac warnings when building intl >>> >>> Modified: >>> lyx-devel/trunk/development/cmake/intl/libgnuintl.h.cmake >>> >>> Modified: lyx-devel/trunk/development/cmake/intl/libgnuintl.h.cmake >>> =========================================================================== >>> === --- lyx-devel/trunk/development/cmake/intl/libgnuintl.h.cmake Sat May >>> 14 14:26:11 2011 (r38753) +++ >>> lyx-devel/trunk/development/cmake/intl/libgnuintl.h.cmake Sat May 14 >>> 14:49:26 2011 (r38754) @@ -19,9 +19,7 @@ >>> #ifndef _LIBINTL_H >>> #define _LIBINTL_H 1 >>> >>> -#cmakedefine01 HAVE_POSIX_PRINTF >>> -#cmakedefine01 HAVE_ASPRINTF >>> -#cmakedefine01 HAVE_WPRINTF >>> +#include "config.h" >>> >>> #include<locale.h> >> >> This does not look ok. >> >> What happens with a line like: >> #if HAVE_ASPRINTF >> >> if HAVE_ASPRINTF is undefined? >> >> Kornel >> > > config.h should define HAVE_ASPRINTF, or have I misunderstood your question?
The question was meant the other way around, IMO. Nevertheless, I think it's correct to include config.h. If a system has no asprintf() implementation you'll get /* #undef HAVE_ASPRINTF */ in configCompiler.h which is included in config.h. HAVE_ASPRINTF is undefined then. Stephan
