Debian Bug report logs - #443637
php5: FTBFS: recode extension can not be configured together with: imap mysql

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Daniel Schepler <schepler@math.unipd.it>

To: submit@bugs.debian.org

Subject: php5: FTBFS: recode extension can not be configured together with: imap mysql

Date: Sat, 22 Sep 2007 22:10:39 -0400

Package: php5
Version: 5.2.4-1
Severity: serious

While trying to rebuild php5 against libsnmp15 (on amd64), I got this error
from pbuilder:

...
checking for XSL support... yes, shared
checking for EXSLT support... found
checking for zip archive read/writesupport... yes
checking for the location of zlib... /usr
configure: error: recode extension can not be configured together with: imap mysql
make: *** [configure-apache2-stamp] Error 1

From a quick diff of this build log from a previous successful build log, it
appears that for some reason configure is not being regenerated.
-- 
Daniel Schepler

Message #10 received at 443637@bugs.debian.org (full text, mbox, reply):

From: Damyan Ivanov <dmn@debian.org>

To: 443637@bugs.debian.org

Cc: vorlon@debian.org

Subject: Re: #443637 php5: FTBFS: recode extension can not be configured together with: imap mysql

Date: Sat, 13 Oct 2007 13:27:03 +0300

[Message part 1 (text/plain, inline)]

Hi,

List night debugging of this, with the help of Atomo64, seanius and
vorlon, led to the conclusion that the build system is broken due to the
suhosin.patch & others changing files in incorrect order.

Thing is, that configure needs to be re-generated in order to avoid the
"recode extension can not be configured together with: imap mysql" error
and replace it with just a warning.

On amd64, due to its hi-res stamps, configure is not re-generated and
so we have the error.

OTOH, re-generating configure (and main/php_config.h.in) overwrites the
patches applied to them so it can't be left for build-time.

vorlon promised to take a look. I'd very much like to see this fixed
"properly", which will have educational effect too :)

My ideas for the fix include yet another patch, that is applied before
most of the rest, that applies the changes to configure and
main/php_config.h.in so there won't be a need to re-generate them in
non-controlled environment (a.k.a. buildd). This would require
reordering of the patches, all patches that modify configure
pre-requisites should be applied before the configure patch for example.
As a nice side effect, auto* may be dropped from build-dependencies.

-- 
dam            JabberID: dam@jabber.minus273.org

[signature.asc (application/pgp-signature, inline)]

Message #15 received at 443637@bugs.debian.org (full text, mbox, reply):

From: Damyan Ivanov <dmn@debian.org>

To: 443637@bugs.debian.org

Cc: vorlon@debian.org

Subject: Patch for #443637: FTBFS

Date: Thu, 18 Oct 2007 16:04:47 +0300

[Message part 1 (text/plain, inline)]

tags 443637 patch
thanks

[Cc-ed to vorlon so he can give his expert opinion too :)]

Hi,

I don't attach the patch, since it is 500k bz2. Find it at
http://people.debian.org/~dmn/php5_auto_kung-foo.patch.bz2

Changelog:

 * Avoid re-generation of auto-foo during build
   Closes: #443637: FTBFS: recode extension can not be configured together
   with: imap mysql
   + Drop autoconf and automake from Build-Depends
   + Drop parts of suhoshin.patch fiddling with generated content
   + Add new patch, 999_autoreconf.patch, containing the results of
     autoreconf
   + Remove config.guess in clean
   + added debian/regen-auto-patches.sh to help re-generate
     999_autoreconf.patch on new upstream releases
   + Patch PHP_EXTRA_VERSION directly in configure, after all other patches

The whole idea is to avoid any kind of re-generation of autotools files
during the build process. Instead, we re-generate what is needed once
and place the change in a big fat patch, 999_autoreconf.patch.

For this to happen, we also need to drop the part of suhoshin.patch that
changes files that we re-build.

There is a final glitch, handling of PHP_EXTRA_VERSION cannot be
handlied via a patch, since this should reflect whenever the package is
NMU/security uploaded. Thus, configure is hand-patched in the prepare
target (was: configure.in patched in the same manner).

To ease the management of the 999_autoreconf.patch, a new script pops
up, debian/regen-auto-patches.sh. Reading through it should make obvious
what it does.

Build tested on i386 and amd64.

If any part of the above needs clarification, I'd be happy to try to
explain.
-- 
dam            JabberID: dam@jabber.minus273.org

[signature.asc (application/pgp-signature, inline)]

Message #22 received at 443637@bugs.debian.org (full text, mbox, reply):

From: Damyan Ivanov <dmn@debian.org>

To: 443637@bugs.debian.org

Subject: Re: Patch for #443637: FTBFS

Date: Thu, 18 Oct 2007 16:26:24 +0300

[Message part 1 (text/plain, inline)]

-=| Damyan Ivanov, Thu, Oct 18, 2007 at 04:04:47PM +0300 |=-
> Build tested on i386 and amd64.

Almost :/

Part of the install procedure wants config.guess, which I dropped in
clean target as it looked as unneeded.

Either config.guess needs to be restored (and added to
999_autoreconf.patch -- see regen-autoreconf-patches.sh), or, dropped
from install too.

What use would config.guess be?

-- 
dam            JabberID: dam@jabber.minus273.org

[signature.asc (application/pgp-signature, inline)]

Message #29 received at 443637@bugs.debian.org (full text, mbox, reply):

From: Steve Langasek <vorlon@debian.org>

To: Damyan Ivanov <dmn@debian.org>, 443637@bugs.debian.org

Subject: Re: #443637 php5: FTBFS: recode extension can not be configured together with: imap mysql

Date: Sat, 20 Oct 2007 17:08:17 -0700

On Sat, Oct 13, 2007 at 01:27:03PM +0300, Damyan Ivanov wrote:

> List night debugging of this, with the help of Atomo64, seanius and
> vorlon, led to the conclusion that the build system is broken due to the
> suhosin.patch & others changing files in incorrect order.

> Thing is, that configure needs to be re-generated in order to avoid the
> "recode extension can not be configured together with: imap mysql" error
> and replace it with just a warning.

> On amd64, due to its hi-res stamps, configure is not re-generated and
> so we have the error.

> OTOH, re-generating configure (and main/php_config.h.in) overwrites the
> patches applied to them so it can't be left for build-time.

> vorlon promised to take a look. I'd very much like to see this fixed
> "properly", which will have educational effect too :)

> My ideas for the fix include yet another patch, that is applied before
> most of the rest, that applies the changes to configure and
> main/php_config.h.in so there won't be a need to re-generate them in
> non-controlled environment (a.k.a. buildd). This would require
> reordering of the patches, all patches that modify configure
> pre-requisites should be applied before the configure patch for example.
> As a nice side effect, auto* may be dropped from build-dependencies.

Dropping auto* from the build-deps is a nice side-effect, but the not-nice
side effect is the very large patch to the autogenerated files which
results, as you noted in your subsequent message.  I've gone ahead with
applying a different fix to svn, which consists of:

- remove configure and main/php_config.h.in from the suhosin patch
- remove these autogenerated files in the clean target (instead of
  re-running buildconf --force, which is not guaranteed to revert the
  changes), which gives a warning when building the source package but
  otherwise results in a clean diff.gz.

Stowing the autogenerated changes in a Debian patch is also valid, but one
of the concerns I have with that approach is that, especially in a large
team, maintainers may forget to regenerate the configure script after making
changes affecting configure.in or the like.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
vorlon@debian.org                                   http://www.debian.org/

Message #34 received at 443637@bugs.debian.org (full text, mbox, reply):

From: Steve Langasek <vorlon@debian.org>

To: Damyan Ivanov <dmn@debian.org>, 443637@bugs.debian.org

Subject: Re: [php-maint] Bug#443637: Patch for #443637: FTBFS

Date: Sat, 20 Oct 2007 17:12:52 -0700

On Thu, Oct 18, 2007 at 04:26:24PM +0300, Damyan Ivanov wrote:
> -=| Damyan Ivanov, Thu, Oct 18, 2007 at 04:04:47PM +0300 |=-
> > Build tested on i386 and amd64.

> Almost :/

> Part of the install procedure wants config.guess, which I dropped in
> clean target as it looked as unneeded.

> Either config.guess needs to be restored (and added to
> 999_autoreconf.patch -- see regen-autoreconf-patches.sh), or, dropped
> from install too.
> 
> What use would config.guess be?

The upstream install process wants to include this file as part of the
phpize dev bits, so that it can copy it into the phpized sources.  We don't
actually ship it in php5-dev, we ship a symlink to the autotools-dev copy
instead so that we don't have to worry about propagating changes to the
file or duplicating contents of binary files.

Doesn't matter in practice anyway in the committed version of the patch,
which re-autotools at build time.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
vorlon@debian.org                                   http://www.debian.org/

Message #39 received at 443637-close@bugs.debian.org (full text, mbox, reply):

From: sean finney <seanius@debian.org>

To: 443637-close@bugs.debian.org

Subject: Bug#443637: fixed in php5 5.2.4-2

Date: Wed, 24 Oct 2007 20:53:02 +0000

Source: php5
Source-Version: 5.2.4-2

We believe that the bug you reported is fixed in the latest version of
php5, which is due to be installed in the Debian FTP archive:

libapache2-mod-php5_5.2.4-2_amd64.deb
  to pool/main/p/php5/libapache2-mod-php5_5.2.4-2_amd64.deb
php-pear_5.2.4-2_all.deb
  to pool/main/p/php5/php-pear_5.2.4-2_all.deb
php5-cgi_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-cgi_5.2.4-2_amd64.deb
php5-cli_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-cli_5.2.4-2_amd64.deb
php5-common_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-common_5.2.4-2_amd64.deb
php5-curl_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-curl_5.2.4-2_amd64.deb
php5-dev_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-dev_5.2.4-2_amd64.deb
php5-gd_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-gd_5.2.4-2_amd64.deb
php5-gmp_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-gmp_5.2.4-2_amd64.deb
php5-imap_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-imap_5.2.4-2_amd64.deb
php5-interbase_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-interbase_5.2.4-2_amd64.deb
php5-ldap_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-ldap_5.2.4-2_amd64.deb
php5-mcrypt_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-mcrypt_5.2.4-2_amd64.deb
php5-mhash_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-mhash_5.2.4-2_amd64.deb
php5-mysql_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-mysql_5.2.4-2_amd64.deb
php5-odbc_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-odbc_5.2.4-2_amd64.deb
php5-pgsql_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-pgsql_5.2.4-2_amd64.deb
php5-pspell_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-pspell_5.2.4-2_amd64.deb
php5-recode_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-recode_5.2.4-2_amd64.deb
php5-snmp_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-snmp_5.2.4-2_amd64.deb
php5-sqlite_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-sqlite_5.2.4-2_amd64.deb
php5-sybase_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-sybase_5.2.4-2_amd64.deb
php5-tidy_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-tidy_5.2.4-2_amd64.deb
php5-xmlrpc_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-xmlrpc_5.2.4-2_amd64.deb
php5-xsl_5.2.4-2_amd64.deb
  to pool/main/p/php5/php5-xsl_5.2.4-2_amd64.deb
php5_5.2.4-2.diff.gz
  to pool/main/p/php5/php5_5.2.4-2.diff.gz
php5_5.2.4-2.dsc
  to pool/main/p/php5/php5_5.2.4-2.dsc
php5_5.2.4-2_all.deb
  to pool/main/p/php5/php5_5.2.4-2_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 443637@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
sean finney <seanius@debian.org> (supplier of updated php5 package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Wed, 24 Oct 2007 21:51:14 +0200
Source: php5
Binary: php5-gd php5-ldap php5 php5-xmlrpc php5-pspell libapache2-mod-php5 php5-xsl php5-cgi php-pear php5-tidy php5-pgsql php5-cli php5-recode php5-mhash php5-sybase php5-curl php5-gmp php5-odbc php5-mcrypt php5-mysql php5-common php5-imap php5-snmp php5-dev php5-sqlite php5-interbase
Architecture: source amd64 all
Version: 5.2.4-2
Distribution: unstable
Urgency: low
Maintainer: Debian PHP Maintainers <pkg-php-maint@lists.alioth.debian.org>
Changed-By: sean finney <seanius@debian.org>
Description: 
 libapache2-mod-php5 - server-side, HTML-embedded scripting language (apache 2 module)
 php-pear   - PEAR - PHP Extension and Application Repository
 php5       - server-side, HTML-embedded scripting language (meta-package)
 php5-cgi   - server-side, HTML-embedded scripting language (CGI binary)
 php5-cli   - command-line interpreter for the php5 scripting language
 php5-common - Common files for packages built from the php5 source
 php5-curl  - CURL module for php5
 php5-dev   - Files for PHP5 module development
 php5-gd    - GD module for php5
 php5-gmp   - GMP module for php5
 php5-imap  - IMAP module for php5
 php5-interbase - interbase/firebird module for php5
 php5-ldap  - LDAP module for php5
 php5-mcrypt - MCrypt module for php5
 php5-mhash - MHASH module for php5
 php5-mysql - MySQL module for php5
 php5-odbc  - ODBC module for php5
 php5-pgsql - PostgreSQL module for php5
 php5-pspell - pspell module for php5
 php5-recode - recode module for php5
 php5-snmp  - SNMP module for php5
 php5-sqlite - SQLite module for php5
 php5-sybase - Sybase / MS SQL Server module for php5
 php5-tidy  - tidy module for php5
 php5-xmlrpc - XML-RPC module for php5
 php5-xsl   - XSL module for php5
Closes: 433736 439286 440624 443637
Changes: 
 php5 (5.2.4-2) unstable; urgency=low
 .
   [ sean finney ]
   * for posterity revised previous changelog to reference the CVE id's
     of security issues resolved by the latest upstream release.
   * lintian: use debian/compat instead of DH_COMPAT in debian/rules.
   * lintian: use source:Version and binary:Version where appropriate,
     instead of Source-Version
   * lintian: remove a couple pieces of cruft in the changelog that were causing
     false-postive wrong-bug-number-in-closes, but were generally useless
     anyway.
 .
   [ Raphael Geissert ]
   * Using test-results.txt as a target
   * cronjob now checks for existance of /usr/lib/php5/maxlifetime (Closes: #439286)
   * Fixed memory limit of 1232M in php.ini for cli (Closes: #440624)
   * Build the interbase extension using firebird2.0-dev (Closes: #433736)
   * Unapply patches with debian/rules clean
 .
   [ Steve Langasek ]
   * Don't patch configure or php_config.h.in in suhosin.patch, as these are
     auto-generated and including them in the patch results in a race
     condition for the necessary build-time regeneration.  Thanks to Daniel
     Schepler for reporting, and to Damyan Ivanov for helping to sort out the
     fix.  Closes: #443637.
   * Also remove the modified auto-generated files in the clean target,
     which triggers a warning about disappearing files when building the
     source package but avoids carrying irrelevant diffs to these files
     in the Debian diff.
   * Now that the testsuite is being run at build time, test failures cause
     a bunch of junk files to be left around in the Debian diff.  So clean up
     several false-positive failures:
     - 052-phpinfo_no_configure.patch: we're patching the output of phpinfo(),
       so patch the test as well
     - fix_broken_upstream_tests.patch: use a local directory for tests that
       use sessions, skip the phpinfo test after all because it doesn't appear
       to be compatible with current testsuite behavior, and disable the
       moneyformat test if en_US locale is not available.
     There are still several other failing tests, but these are not false
     positives and remain enabled pending investigation.
Files: 
 9392545cd300e59f8d317094141da8ce 2001 web optional php5_5.2.4-2.dsc
 d62d2e300c83f9aa4ff9a4b6414f9a4b 121383 web optional php5_5.2.4-2.diff.gz
 158d1b7dbef4d3ce5cb5229deadffe2c 315444 web optional php5-common_5.2.4-2_amd64.deb
 d2fbd508c800df40cc404f68f5aba550 2682322 web optional libapache2-mod-php5_5.2.4-2_amd64.deb
 5d69f5577d6936b2ad95a0be3b6adc2e 5214976 web optional php5-cgi_5.2.4-2_amd64.deb
 b03dc8ba9336f58f879fa4ac86687fc4 2628220 web optional php5-cli_5.2.4-2_amd64.deb
 0551a3c0a32b7f3bd6349944bac301ad 364764 devel optional php5-dev_5.2.4-2_amd64.deb
 4f518e3dc8590bd007ea7285a7947ff8 24744 web optional php5-curl_5.2.4-2_amd64.deb
 af227b7f4e1e2163b6e635af562973d4 36784 web optional php5-gd_5.2.4-2_amd64.deb
 10ff575f5003802698f4a76175ab8318 17350 web optional php5-gmp_5.2.4-2_amd64.deb
 2324b39e23703f6aa76bf38d7f360ca4 37494 web optional php5-imap_5.2.4-2_amd64.deb
 70a7c511be85626244b460eda3ccd229 47382 web optional php5-interbase_5.2.4-2_amd64.deb
 7eb4e4c0fc713952847e777fcfa03dee 19810 web optional php5-ldap_5.2.4-2_amd64.deb
 1b077244836ee3d3e9ac17824cee0bfa 13882 web optional php5-mcrypt_5.2.4-2_amd64.deb
 4ac63cd60280bac3f755484404a7e64d 5290 web optional php5-mhash_5.2.4-2_amd64.deb
 b198e39c08f027051056a532feee8b59 73006 web optional php5-mysql_5.2.4-2_amd64.deb
 3e2305989f457d8c9aca2e77b0b588e9 37254 web optional php5-odbc_5.2.4-2_amd64.deb
 36e2c1c10f85e5e4178dc2737d01f8eb 55680 web optional php5-pgsql_5.2.4-2_amd64.deb
 29481f423108c7cf6da1dd2ae1c49e40 9386 web optional php5-pspell_5.2.4-2_amd64.deb
 646f8eb62a887c22604e79d5b4509984 4910 web optional php5-recode_5.2.4-2_amd64.deb
 f0aec4cc65089c4efdc261d3a3747839 12132 web optional php5-snmp_5.2.4-2_amd64.deb
 dd27922f616f98ab6305723968c58813 39084 web optional php5-sqlite_5.2.4-2_amd64.deb
 08b889036335f665cbe05214789f435b 27732 web optional php5-sybase_5.2.4-2_amd64.deb
 916c130305b5a28598ab70aa92e3d496 17676 web optional php5-tidy_5.2.4-2_amd64.deb
 6867f76e9a398974f87536c35cec7b4f 39796 web optional php5-xmlrpc_5.2.4-2_amd64.deb
 717bbd1c6b9c9a935b9800c51e4d39d1 13344 web optional php5-xsl_5.2.4-2_amd64.deb
 cd417e54b63102c0b71c2bc58b6efa0e 1032 web optional php5_5.2.4-2_all.deb
 94ba3ab82464c96011d4a28b508c165a 359830 web optional php-pear_5.2.4-2_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHH6j3ynjLPm522B0RAknvAJ9LrzWkWQxIG7BA8u5V32CZ4/23sgCfa0N8
Q5Wm7hHjxInSGjJv05aEaHM=
=9F1g
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.