>> Good, then I'll try to do the change for scons. > It seems to be there already: > (conf.CheckIconvConst(), > 'ICONV_CONST', > 'Define as const if the declaration of iconv() needs > const.', > '#define ICONV_CONST', > '#define ICONV_CONST const', > ),
The test is for intl/config.h, not for common/config.h. That is why you can not see it when nls is not supported. I can of course apply the attached patch and add the test to common/config.h, but is intl always needed for unicode to work? If so, more complicated scenario should be used. Namely, disable unicode when gettext is not available, conditionally include intl/config.h in common/config.h etc. Opinion? Bo
scons-iconv.diff
Description: Binary data
