[Buildroot] [PATCH] php: fix wrong -L and -I paths added by iconv tests
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sat Apr 12 11:14:21 UTC 2014
Dear Max Filippov,
On Sat, 12 Apr 2014 14:37:02 +0400, Max Filippov wrote:
> The commit f2a2c4cce "php: fix iconv related build failure" have removed
> logic that used to detect the correct PHP_ICONV_PREFIX with an
> assumption that "ICONV_DIR is fine". Currently the ICONV_DIR is only
> fine when BR2_PACKAGE_LIBICONV is set to 'y', otherwise php is
> configured with plain --with-iconv. This results in empty
> PHP_ICONV_PREFIX and -L/lib and -I/include added to CFLAGS and LDFLAGS
> respectively, which leads to build failures when e.g. /lib contains
> files with the same names as the toolchain sysroot needed for linking.
>
> Fix that by always specifying --with-iconv=$(STAGING_DIR)/usr when iconv
> is configured for PHP.
>
> Fixes: http://autobuild.buildroot.net/results/959/959b77fa2c1f13b1958b234803437e09734e882e/
>
> Cc: Gustavo Zacarias <gustavo at zacarias.com.ar>
> Cc: Peter Korsgaard <peter at korsgaard.com>
> Signed-off-by: Max Filippov <jcmvbkbc at gmail.com>
> ---
> package/php/php.mk | 4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
I remember trying this possible fix the other day before replying to
you, and it didn't work: the iconv detection was failing. I'll
reproduce and get back to you with more details.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list