# autoconf version check on trunk? - Thread: https://externals.io/message/46201 - Original archive: https://news-web.php.net/php.internals/46201 - Started by Arvind Srinivasan on 2009-11-27T08:56:25+00:00 - Messages: 9 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #46201 — Arvind Srinivasan — 2009-11-27T08:56:25+00:00 URL: https://externals.io/message/46201#46201 There was some discussion on the version (2.13 vs newer) of autoconf to use for trunk but I'm not sure whether there was any consensus. It looks like autoconf2.13 can't be used with trunk and therefore the autoconf version check inside build/buildcheck.sh needs to be updated. I get the following error when I try to configure trunk with autoconf 2.13 (this is on Ubuntu [Hardy]). autoconf2.61 works fine. Checked out revision 291346. % ./buildconf buildconf: checking installation... buildconf: autoconf version 2.13 (ok) rebuilding aclocal.m4 rebuilding configure rebuilding acconfig.h rebuilding main/php_config.h.in % ./configure --prefix=/home/arvi/php-src-build --enable-debug (stuff deleted) checking whether to enable UCD SNMP hack... no checking whether to enable SOAP support... no checking whether to enable sockets support... no checking whether zend_object_value is packed... yes checking for sqlite support... yes checking whether to disable UTF-8 support in libsqlite (charset changes to ISO-8859-1)... yes checking for PDO includes... (cached) /home/arvi/php-scratch/ext checking for lemon... no configure: warning: lemon versions supported for regeneration of libsqlite parsers: 1.0 (found: none). checking size of char *... 4 checking for usleep... (cached) yes checking for nanosleep... (cached) yes checking for time.h... (cached) yes ./configure: line 81214: syntax error at line 81217: `fi' unexpected --- ## #46202 — Jani Taskinen — 2009-11-27T09:26:42+00:00 URL: https://externals.io/message/46201#46202 · In reply to: #46201 It works fine as soon as I find the typo Rasmus has done with his fixes for autoconf > 2.13.. --Jani > There was some discussion on the version (2.13 vs newer) of autoconf > to use for trunk but I'm not sure whether there was any consensus. > > It looks like autoconf2.13 can't be used with trunk and therefore the > autoconf version check inside build/buildcheck.sh needs to be updated. > > I get the following error when I try to configure trunk with autoconf > 2.13 (this is on Ubuntu [Hardy]). autoconf2.61 works fine. > > > Checked out revision 291346. > % ./buildconf > buildconf: checking installation... > buildconf: autoconf version 2.13 (ok) > rebuilding aclocal.m4 > rebuilding configure > rebuilding acconfig.h > rebuilding main/php_config.h.in > > % ./configure --prefix=/home/arvi/php-src-build --enable-debug > > (stuff deleted) > > checking whether to enable UCD SNMP hack... no > checking whether to enable SOAP support... no > checking whether to enable sockets support... no > checking whether zend_object_value is packed... yes > checking for sqlite support... yes > checking whether to disable UTF-8 support in libsqlite (charset > changes to ISO-8859-1)... yes > checking for PDO includes... (cached) /home/arvi/php-scratch/ext > checking for lemon... no > configure: warning: lemon versions supported for regeneration of > libsqlite parsers: 1.0 (found: none). > checking size of char *... 4 > checking for usleep... (cached) yes > checking for nanosleep... (cached) yes > checking for time.h... (cached) yes > /configure: line 81214: syntax error at line 81217: `fi' unexpected --- ## #46204 — Jani Taskinen — 2009-11-27T11:08:37+00:00 URL: https://externals.io/message/46201#46204 · In reply to: #46202 Make that me. It's either my upgrade of libtool or combined changes by me and Rasmus. And it affects ALL branches.. :( I'm investigating this bug atm, stay tuned. --Jani > It works fine as soon as I find the typo Rasmus has done with his fixes > for autoconf > 2.13.. > > --Jani > > > > […] --- ## #46205 — Jani Taskinen — 2009-11-27T11:32:08+00:00 URL: https://externals.io/message/46201#46205 · In reply to: #46204 This is caused by the divert() patch Rasmus committed. Nice work. I'll try figure out how to do it properly. --Jani > Make that me. It's either my upgrade of libtool or combined changes by > me and Rasmus. And it affects ALL branches.. :( > > I'm investigating this bug atm, stay tuned. > > --Jani > > > > […] --- ## #46206 — Rasmus Lerdorf — 2009-11-27T11:38:26+00:00 URL: https://externals.io/message/46201#46206 · In reply to: #46205 Jani Taskinen wrote: > This is caused by the divert() patch Rasmus committed. Nice work. > I'll try figure out how to do it properly. The easy way would be to get rid of the diverts completely. They don't do anything in autoconf2.13 anyway, so I don't really see why they were added in the first place. -Rasmus --- ## #46208 — Rasmus Lerdorf — 2009-11-27T12:30:52+00:00 URL: https://externals.io/message/46201#46208 · In reply to: #46205 Jani Taskinen wrote: > This is caused by the divert() patch Rasmus committed. Nice work. > I'll try figure out how to do it properly. Getting rid of just the diverts in ext/standard/config.m4 fixes this for me. There is still an unrelated libtool/autoconf-2.13 issue related to finding ld though. Not sure what that is about yet. -Rasmus --- ## #46209 — Jani Taskinen — 2009-11-27T12:44:35+00:00 URL: https://externals.io/message/46201#46209 · In reply to: #46208 > Jani Taskinen wrote: > > […] > > Getting rid of just the diverts in ext/standard/config.m4 fixes this for > me. There is still an unrelated libtool/autoconf-2.13 issue related to > finding ld though. Not sure what that is about yet. How about forget about it and stick with autoconf 2.13 which actually works and does what we need? Try ./configure --help once with the 2.6x generated one.. --Jani --- ## #46210 — Ilia Alshanetsky — 2009-11-27T12:49:26+00:00 URL: https://externals.io/message/46201#46210 · In reply to: #46208 Rasmus, Do you think this is an easy fix, or do we need to go back pre-patch code for 5.2.12? The other option is to generate the build on a machine with the later autoconf, 2.59 seems to work fine. > Jani Taskinen wrote: > > […] > > Getting rid of just the diverts in ext/standard/config.m4 fixes this for > me. There is still an unrelated libtool/autoconf-2.13 issue related to > finding ld though. Not sure what that is about yet. > > -Rasmus --- ## #46211 — Jani Taskinen — 2009-11-27T13:02:32+00:00 URL: https://externals.io/message/46201#46211 · In reply to: #46210 As I mentioned in IRC, this is now fixed for PHP_5_2. Rasmus, please don't commit any more "fixes" into that branch. Experiments should be done in trunk only. :) --Jani > Rasmus, > > Do you think this is an easy fix, or do we need to go back pre-patch code for 5.2.12? The other option is to generate the build on a machine with the later autoconf, 2.59 seems to work fine. > > > > > > […]