[Buildroot] [autobuild.buildroot.net] Build results for 2014-04-07

[Max Filippov]

Hi Thomas,

On Fri, Apr 11, 2014 at 1:41 AM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Dear Max Filippov,
>
> On Wed, 9 Apr 2014 01:51:08 +0400, Max Filippov wrote:
>
>> >> >     xtensa |                     php-5.5.11 | NOK | http://autobuild.buildroot.net/results/959b77fa2c1f13b1958b234803437e09734e882e/
>> >>
>> >> Couldn't reproduce. But I had to disable BR2_PACKAGE_HOST_SAM_BA,
>> >> otherwise it refused to build with the following message:
>> >>
>> >> Your Buildroot configuration uses pre-built tools for the x86 architecture,
>> >> but your build machine uses the x86-64 architecture without the 32
>> >> bits compatibility library.
>> >
>> > I think we can safely assume that the problem is not due to
>> > BR2_PACKAGE_HOST_SAM_BA. I'll try to reproduce the same build failure
>> > on the autobuilder, and gather the config.log file of php. I'll let you
>> > know.
>>
>> The tail of the failing build log:
>>
>> ...
>> checking for MySQL support... yes
>> checking for specified location of the MySQL UNIX socket... no
>> checking for mysql_close in -lmysqlclient... no
>> checking for mysql_error in -lmysqlclient... no
>> configure: error: mysql configure failed. Please check config.log for
>> more information.
>> make: *** [/home/test/test/2/output/build/php-5.5.11/.stamp_configured] Error 1
>
> Ok, I reproduced the problem. The config.log of php contains the
> following problem:
>
> configure:58794: result: no
> configure:59004: checking for mysql_error in -lmysqlclient
> configure:59029: /home/test/outputs/959b77fa2c1f13b1958b234803437e09734e882e/output/host/usr/bin/xtensa-buildroot-linux-uclibc-gcc -o conftest -I/include -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURC
> E -D_FILE_OFFSET_BITS=64 -mlongcalls -mtext-section-literals -pipe -Os  -fvisibility=hidden -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -L/home/test/outputs/959b77fa2c1f13b1
> 958b234803437e09734e882e/output/host/usr/xtensa-buildroot-linux-uclibc/sysroot/usr/lib/mysql -L/home/test/outputs/959b77fa2c1f13b1958b234803437e09734e882e/output/host/usr/xtensa-buildroot-linux-
> uclibc/sysroot/usr/lib -L/lib  -L/home/test/outputs/959b77fa2c1f13b1958b234803437e09734e882e/output/host/usr/xtensa-buildroot-linux-uclibc/sysroot/usr/lib -L/home/test/outputs/959b77fa2c1f13b195
> 8b234803437e09734e882e/output/host/usr/xtensa-buildroot-linux-uclibc/sysroot/usr conftest.c -lmysqlclient  -lz -lintl -lcrypto -lssl -lcrypto -lcurl -lz -lcrypto -lssl -lcrypto -lm  -lxml2 -lz -
> lm -ldl -lcurl >&5
> /lib/libgcc_s.so.1: file not recognized: File format not recognized
> collect2: error: ld returned 1 exit status
>
> The problem is due to the compilation command line containing -L/lib,
> which is wrong when cross-compiling.

Thanks for the information. I don't have /lib/libgcc_s.so.1 on my build system,
so it doesn't break here.
Apparently this happens when

BR2_PACKAGE_PHP_EXT_ICONV = y and
BR2_PACKAGE_LIBICONV is unset.

In this case buildroot calls

configure --with-iconv

for php, omitting LIBICONV dir, and php adds system dirs to -I and -L
(though I'm not sure it does that correctly).

Looks like we need to always select BR2_PACKAGE_LIBICONV when
BR2_PACKAGE_PHP_EXT_ICONV is selected, i.e.:

diff --git a/package/php/Config.ext b/package/php/Config.ext
index abf4993..1a46061 100644
--- a/package/php/Config.ext
+++ b/package/php/Config.ext
@@ -152,7 +152,7 @@ comment "Gettext support needs a toolchain w/ wchar"

 config BR2_PACKAGE_PHP_EXT_ICONV
        bool "iconv"
-       select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
+       select BR2_PACKAGE_LIBICONV
        help
          iconv character set conversion support

diff --git a/package/php/php.mk b/package/php/php.mk
index ba1200b..8a147c5 100644
--- a/package/php/php.mk
+++ b/package/php/php.mk
@@ -108,12 +108,8 @@ ifeq ($(BR2_PACKAGE_PHP_EXT_GETTEXT),y)
 endif

 ifeq ($(BR2_PACKAGE_PHP_EXT_ICONV),y)
-ifeq ($(BR2_PACKAGE_LIBICONV),y)
        PHP_CONF_OPT += --with-iconv=$(STAGING_DIR)/usr
        PHP_DEPENDENCIES += libiconv
-else
-       PHP_CONF_OPT += --with-iconv
-endif
 endif

 ifeq ($(BR2_PACKAGE_PHP_EXT_INTL),y)

-- 
Thanks.
-- Max