swig1.3: ftbfs [sparc] too few arguments to function `zend_rsrc_list_get_rsrc_type'

[Debian Bug Importer]

Automatically imported from Debian bug report #294499 http://bugs.debian.org/294499

[Kurt Roeckx (kurt-roeckx)]

severity 294499 serious
thanks

It's failing with the same error on all arches.

Kurt

[Debian Bug Importer (debzilla)]

Automatically imported from Debian bug report #294499 http://bugs.debian.org/294499

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Wed, 09 Feb 2005 16:24:44 -0800
From: Blars Blarson <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: swig1.3: ftbfs [sparc] too few arguments to function `zend_rsrc_list_get_rsrc_type'

Package: swig1.3
Version: 1.3.22-6
Severity: normal

swig1.3 fails to build from source on sparc, duplicated on sparc pbuilder.

 gcc -DHAVE_CONFIG_H -I. -I. -I../Source/Include -I/usr/include/php4 -I/usr/include/php4/main -I/usr/include/php4/Zend -I/usr/include/php4/TSRM -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -MT libswigphp4_la-libphp4.lo -MD -MP -MF .deps/libswigphp4_la-libphp4.Tpo -c libphp4.c -fPIC -DPIC -o .libs/libswigphp4_la-libphp4.o
libphp4.c: In function `SWIG_ZTS_ConvertResourcePtr':
libphp4.c:535: error: too few arguments to function `zend_rsrc_list_get_rsrc_type'
make[2]: *** [libswigphp4_la-libphp4.lo] Error 1
make[2]: Leaving directory `/tmp/buildd/swig1.3-1.3.22/Runtime'
make[1]: *** [runtime] Error 2

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Fri, 11 Feb 2005 21:01:04 +0100
From: Kurt Roeckx <email address hidden>
To: <email address hidden>
Cc: <email address hidden>
Subject: Re: swig1.3: FTBFS: too few arguments to function `zend_rsrc_list_get_rsrc_type'

severity 294499 serious
thanks

It's failing with the same error on all arches.

Kurt

[Torsten Landschoff (torsten)]

On Wed, Feb 09, 2005 at 04:24:44PM -0800, Blars Blarson wrote:

> swig1.3 fails to build from source on sparc, duplicated on sparc pbuilder.
>
> gcc -DHAVE_CONFIG_H -I. -I. -I../Source/Include -I/usr/include/php4 -I/usr/include/php4/main -I/usr/include/php4/Zend -I/usr/include/php4/TSRM -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -MT libswigphp4_la-libphp4.lo -MD -MP -MF .deps/libswigphp4_la-libphp4.Tpo -c libphp4.c -fPIC -DPIC -o .libs/libswigphp4_la-libphp4.o
> libphp4.c: In function `SWIG_ZTS_ConvertResourcePtr':
> libphp4.c:535: error: too few arguments to function `zend_rsrc_list_get_rsrc_type'

Okay, that was part of the runtime libraries. They are gone (replaced)
in swig 1.3.24 which is already packaged but I am waiting for dependent
packages to repair the no-runtime-anymore breakage.

Should be a non-issue mid of next week.

Greetings

 Torsten

[Torsten Landschoff (t-landschoff)]

tags 294499 pending

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Sat, 12 Feb 2005 19:47:11 +0100
From: Torsten Landschoff <email address hidden>
To: Blars Blarson <email address hidden>, <email address hidden>
Subject: Re: Bug#294499: swig1.3: ftbfs [sparc] too few arguments to function
 `zend_rsrc_list_get_rsrc_type'

--Y5rl02BVI9TCfPar
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Feb 09, 2005 at 04:24:44PM -0800, Blars Blarson wrote:
=20
> swig1.3 fails to build from source on sparc, duplicated on sparc pbuilder.
>=20
> gcc -DHAVE_CONFIG_H -I. -I. -I../Source/Include -I/usr/include/php4 -I/u=
sr/include/php4/main -I/usr/include/php4/Zend -I/usr/include/php4/TSRM -D_R=
EENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN -fno-strict-aliasing -I=
/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=3D64 -MT libswig=
php4_la-libphp4.lo -MD -MP -MF .deps/libswigphp4_la-libphp4.Tpo -c libphp4.=
c -fPIC -DPIC -o .libs/libswigphp4_la-libphp4.o
> libphp4.c: In function `SWIG_ZTS_ConvertResourcePtr':
> libphp4.c:535: error: too few arguments to function `zend_rsrc_list_get_r=
src_type'

Okay, that was part of the runtime libraries. They are gone (replaced)
in swig 1.3.24 which is already packaged but I am waiting for dependent=20
packages to repair the no-runtime-anymore breakage.

Should be a non-issue mid of next week.

Greetings

 Torsten

--Y5rl02BVI9TCfPar
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFCDk8vdQgHtVUb5EcRAkhZAJ934f4bwhE0BgF85YbFTftRDAiQmACaAjhH
glkmgMJUJGMHPRAkOIGKVo0=
=duBn
-----END PGP SIGNATURE-----

--Y5rl02BVI9TCfPar--

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Sat, 12 Feb 2005 19:49:10 +0100
From: Torsten Landschoff <email address hidden>
To: <email address hidden>
Subject: tagging 294499

tags 294499 pending

[Olly Betts (ojwb)]

On Sat, Feb 12, 2005 at 07:47:11PM +0100, Torsten Landschoff wrote:
> On Wed, Feb 09, 2005 at 04:24:44PM -0800, Blars Blarson wrote:
>
> > swig1.3 fails to build from source on sparc, duplicated on sparc pbuilder.
> >
> > gcc -DHAVE_CONFIG_H -I. -I. -I../Source/Include -I/usr/include/php4 -I/usr/include/php4/main -I/usr/include/php4/Zend -I/usr/include/php4/TSRM -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -MT libswigphp4_la-libphp4.lo -MD -MP -MF .deps/libswigphp4_la-libphp4.Tpo -c libphp4.c -fPIC -DPIC -o .libs/libswigphp4_la-libphp4.o
> > libphp4.c: In function `SWIG_ZTS_ConvertResourcePtr':
> > libphp4.c:535: error: too few arguments to function `zend_rsrc_list_get_rsrc_type'
>
> Okay, that was part of the runtime libraries. They are gone (replaced)
> in swig 1.3.24 which is already packaged but I am waiting for dependent
> packages to repair the no-runtime-anymore breakage.
>
> Should be a non-issue mid of next week.

Actually, the same problem is still present in PHP wrapper code generated by
SWIG, so I suspect this will move from a FTBFS for SWIG to a FTBFS for any
package which use SWIG to generate PHP bindings. This problem has been there
all along. The reason we're only now seeing it is that the Debian php4 package
recently enabled thread safety in Zend:

  php4 (4:4.3.10-3) unstable; urgency=medium

  [snip]
    * Enable Zend Thread Safety for all SAPIs, meaning that our modules
      are now compiled for ZTS APIs as well. (closes: #278212, #264015)
      - Make sure caudium-php4 now provides phpapi-$(ver), and modules can
        be configured with the caudium SAPI.
      - Add 039-reentrant_libs.patch to link to the reentrant versions of
        libldap and libmysqlclient
  [snip]

   -- Adam Conrad <[e-mail]> Sun, 6 Feb 2005 05:32:11 -0700

I've attached a patch which fixes this.

When thread safety is enabled, zend_rsrc_list_get_rsrc_type now expects more
than one argument, and there's some standard macro magic to deal with this.
It's used throughout the patched file - it's just missing in this one call
(when thread safety is off, TSRMLS_CC is empty, which is why it's worked up
until now).

Cheers,
    Olly

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Mon, 21 Feb 2005 07:39:53 +0000
From: Olly Betts <email address hidden>
To: Torsten Landschoff <email address hidden>
Cc: Blars Blarson <email address hidden>, <email address hidden>
Subject: Re: Bug#294499: swig1.3: ftbfs [sparc] too few arguments to function
 `zend_rsrc_list_get_rsrc_type'

--5I6of5zJg18YgZEa
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Sat, Feb 12, 2005 at 07:47:11PM +0100, Torsten Landschoff wrote:
> On Wed, Feb 09, 2005 at 04:24:44PM -0800, Blars Blarson wrote:
>
> > swig1.3 fails to build from source on sparc, duplicated on sparc pbuilder.
> >
> > gcc -DHAVE_CONFIG_H -I. -I. -I../Source/Include -I/usr/include/php4 -I/usr/include/php4/main -I/usr/include/php4/Zend -I/usr/include/php4/TSRM -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -MT libswigphp4_la-libphp4.lo -MD -MP -MF .deps/libswigphp4_la-libphp4.Tpo -c libphp4.c -fPIC -DPIC -o .libs/libswigphp4_la-libphp4.o
> > libphp4.c: In function `SWIG_ZTS_ConvertResourcePtr':
> > libphp4.c:535: error: too few arguments to function `zend_rsrc_list_get_rsrc_type'
>
> Okay, that was part of the runtime libraries. They are gone (replaced)
> in swig 1.3.24 which is already packaged but I am waiting for dependent
> packages to repair the no-runtime-anymore breakage.
>
> Should be a non-issue mid of next week.

Actually, the same problem is still present in PHP wrapper code generated by
SWIG, so I suspect this will move from a FTBFS for SWIG to a FTBFS for any
package which use SWIG to generate PHP bindings. This problem has been there
all along. The reason we're only now seeing it is that the Debian php4 package
recently enabled thread safety in Zend:

  php4 (4:4.3.10-3) unstable; urgency=medium

  [snip]
    * Enable Zend Thread Safety for all SAPIs, meaning that our modules
      are now compiled for ZTS APIs as well. (closes: #278212, #264015)
      - Make sure caudium-php4 now provides phpapi-$(ver), and modules can
        be configured with the caudium SAPI.
      - Add 039-reentrant_libs.patch to link to the reentrant versions of
        libldap and libmysqlclient
  [snip]

   -- Adam Conrad <[e-mail]> Sun, 6 Feb 2005 05:32:11 -0700

I've attached a patch which fixes this.

When thread safety is enabled, zend_rsrc_list_get_rsrc_type now expects more
than one argument, and there's some standard macro magic to deal with this.
It's used throughout the patched file - it's just missing in this one call
(when thread safety is off, TSRMLS_CC is empty, which is why it's worked up
until now).

Cheers,
    Olly

--5I6of5zJg18YgZEa
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="swig-fix-for-threadsafe-ZTS.patch"

--- Lib/php4/php4run.swg.orig Sat Oct 16 19:23:51 2004
+++ Lib/php4/php4run.swg Mon Feb 21 07:08:22 2005
@@ -195,7 +195,7 @@
   p = value->ptr;
   if (type==-1) return -1;

- type_name=zend_rsrc_list_get_rsrc_type(z->value.lval);
+ type_name=zend_rsrc_list_get_rsrc_type(z->value.lval TSRMLS_CC);

   return SWIG_ZTS_ConvertResourceData(p,type,type_...

[Torsten Landschoff (t-landschoff)]

tags 294499 + patch upstream

[Torsten Landschoff (t-landschoff)]

tags 294499 - pending

[Torsten Landschoff (torsten)]

Hi there,

Seems like the wrapper generation for php4 is a big buggy with Zend
thread safety enabled. The following patch courtesy of Olly Betts should
fix this:

--- Lib/php4/php4run.swg.orig Sat Oct 16 19:23:51 2004
+++ Lib/php4/php4run.swg Mon Feb 21 07:08:22 2005
@@ -195,7 +195,7 @@
   p = value->ptr;
   if (type==-1) return -1;

- type_name=zend_rsrc_list_get_rsrc_type(z->value.lval);
+ type_name=zend_rsrc_list_get_rsrc_type(z->value.lval TSRMLS_CC);

   return SWIG_ZTS_ConvertResourceData(p,type,type_name,ptr,ty TSRMLS_CC);
 }

For more information please have a look at

 http://bugs.debian.org/294499

Thanks

 Torsten

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Mon, 21 Feb 2005 11:52:28 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: tagging 294499

tags 294499 + patch upstream

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Mon, 21 Feb 2005 11:52:34 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: tagging 294499

tags 294499 - pending

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Mon, 21 Feb 2005 11:56:27 +0100
From: Torsten Landschoff <email address hidden>
To: <email address hidden>
Cc: <email address hidden>
Subject: [PATCH] Bug in PHP4 wrapper generation

Hi there,

Seems like the wrapper generation for php4 is a big buggy with Zend
thread safety enabled. The following patch courtesy of Olly Betts should
fix this:

--- Lib/php4/php4run.swg.orig Sat Oct 16 19:23:51 2004
+++ Lib/php4/php4run.swg Mon Feb 21 07:08:22 2005
@@ -195,7 +195,7 @@
   p = value->ptr;
   if (type==-1) return -1;

- type_name=zend_rsrc_list_get_rsrc_type(z->value.lval);
+ type_name=zend_rsrc_list_get_rsrc_type(z->value.lval TSRMLS_CC);

   return SWIG_ZTS_ConvertResourceData(p,type,type_name,ptr,ty TSRMLS_CC);
 }

For more information please have a look at

 http://bugs.debian.org/294499

Thanks

 Torsten

[wsfulton (wsf)]

Torsten Landschoff wrote:
> Hi there,
>
> Seems like the wrapper generation for php4 is a big buggy with Zend
> thread safety enabled. The following patch courtesy of Olly Betts should
> fix this:
>
> --- Lib/php4/php4run.swg.orig Sat Oct 16 19:23:51 2004
> +++ Lib/php4/php4run.swg Mon Feb 21 07:08:22 2005
> @@ -195,7 +195,7 @@
> p = value->ptr;
> if (type==-1) return -1;
>
> - type_name=zend_rsrc_list_get_rsrc_type(z->value.lval);
> + type_name=zend_rsrc_list_get_rsrc_type(z->value.lval TSRMLS_CC);
>
> return SWIG_ZTS_ConvertResourceData(p,type,type_name,ptr,ty TSRMLS_CC);
> }
>
> For more information please have a look at
>
> http://bugs.debian.org/294499
>
> Thanks
>
> Torsten

I've committed this to the cvs repository, but have no way of checking
it, so please try it out.

William

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Tue, 22 Feb 2005 05:41:16 +0000
From: William S Fulton <email address hidden>
To: Torsten Landschoff <email address hidden>
CC: <email address hidden>, <email address hidden>
Subject: Re: [Swig-dev] [PATCH] Bug in PHP4 wrapper generation

Torsten Landschoff wrote:
> Hi there,
>
> Seems like the wrapper generation for php4 is a big buggy with Zend
> thread safety enabled. The following patch courtesy of Olly Betts should
> fix this:
>
> --- Lib/php4/php4run.swg.orig Sat Oct 16 19:23:51 2004
> +++ Lib/php4/php4run.swg Mon Feb 21 07:08:22 2005
> @@ -195,7 +195,7 @@
> p = value->ptr;
> if (type==-1) return -1;
>
> - type_name=zend_rsrc_list_get_rsrc_type(z->value.lval);
> + type_name=zend_rsrc_list_get_rsrc_type(z->value.lval TSRMLS_CC);
>
> return SWIG_ZTS_ConvertResourceData(p,type,type_name,ptr,ty TSRMLS_CC);
> }
>
> For more information please have a look at
>
> http://bugs.debian.org/294499
>
> Thanks
>
> Torsten

I've committed this to the cvs repository, but have no way of checking
it, so please try it out.

William

[Torsten Landschoff (torsten)]

Hi William,

On Tue, Feb 22, 2005 at 05:41:16AM +0000, William S Fulton wrote:
> >Seems like the wrapper generation for php4 is a big buggy with Zend
> >thread safety enabled. The following patch courtesy of Olly Betts should

> I've committed this to the cvs repository, but have no way of checking
> it, so please try it out.

Just did. Seems like there are a number of additional problems. I tried
wrapping a simple example and it fails with

cdaudio_wrap.c|1059| error: `tsrm_ls' undeclared (first use in this
function)cdaudio_wrap.c|1059| error: (Each undeclared identifier is
reported only once

every time SWIG_ConvertPtr ist called. Which is no wonder since that
macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
TSRMLS_DC).

No idea how to fix this :(

Greetings

 Torsten

[Torsten Landschoff (t-landschoff)]

retitle 294499 php4 wrappers broken for thread-safe php

[Torsten Landschoff (t-landschoff)]

tags 294499 upstream

[Torsten Landschoff (t-landschoff)]

tags 294499 confirmed

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Tue, 22 Feb 2005 10:26:20 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: retitle 294499 to php4 wrappers broken for thread-safe php

retitle 294499 php4 wrappers broken for thread-safe php

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Tue, 22 Feb 2005 10:26:27 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: tagging 294499

tags 294499 upstream

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Tue, 22 Feb 2005 10:26:51 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: tagging 294499

tags 294499 confirmed

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Tue, 22 Feb 2005 10:25:14 +0100
From: Torsten Landschoff <email address hidden>
To: William S Fulton <email address hidden>
Cc: <email address hidden>, <email address hidden>
Subject: Re: [Swig-dev] [PATCH] Bug in PHP4 wrapper generation

Hi William,

On Tue, Feb 22, 2005 at 05:41:16AM +0000, William S Fulton wrote:
> >Seems like the wrapper generation for php4 is a big buggy with Zend
> >thread safety enabled. The following patch courtesy of Olly Betts should

> I've committed this to the cvs repository, but have no way of checking
> it, so please try it out.

Just did. Seems like there are a number of additional problems. I tried
wrapping a simple example and it fails with

cdaudio_wrap.c|1059| error: `tsrm_ls' undeclared (first use in this
function)cdaudio_wrap.c|1059| error: (Each undeclared identifier is
reported only once

every time SWIG_ConvertPtr ist called. Which is no wonder since that
macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
TSRMLS_DC).

No idea how to fix this :(

Greetings

 Torsten

[wsfulton (wsf)]

Torsten Landschoff wrote:
> Hi William,
>
> On Tue, Feb 22, 2005 at 05:41:16AM +0000, William S Fulton wrote:
>>>Seems like the wrapper generation for php4 is a big buggy with Zend
>>>thread safety enabled. The following patch courtesy of Olly Betts should
>
>>I've committed this to the cvs repository, but have no way of checking
>>it, so please try it out.
>
> Just did. Seems like there are a number of additional problems. I tried
> wrapping a simple example and it fails with
>
> cdaudio_wrap.c|1059| error: `tsrm_ls' undeclared (first use in this
> function)cdaudio_wrap.c|1059| error: (Each undeclared identifier is
> reported only once
>
> every time SWIG_ConvertPtr ist called. Which is no wonder since that
> macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
> TSRMLS_DC).
>
> No idea how to fix this :(
>
I'm afraid that the PHP module is under neglect as the PHP maintainer
isn't around anymore. I'll commit any patches until a new volunteer
comes along though.

William

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Tue, 22 Feb 2005 21:00:52 +0000
From: William S Fulton <email address hidden>
To: Torsten Landschoff <email address hidden>
CC: <email address hidden>, <email address hidden>
Subject: Re: [Swig-dev] [PATCH] Bug in PHP4 wrapper generation

Torsten Landschoff wrote:
> Hi William,
>
> On Tue, Feb 22, 2005 at 05:41:16AM +0000, William S Fulton wrote:
>>>Seems like the wrapper generation for php4 is a big buggy with Zend
>>>thread safety enabled. The following patch courtesy of Olly Betts should
>
>>I've committed this to the cvs repository, but have no way of checking
>>it, so please try it out.
>
> Just did. Seems like there are a number of additional problems. I tried
> wrapping a simple example and it fails with
>
> cdaudio_wrap.c|1059| error: `tsrm_ls' undeclared (first use in this
> function)cdaudio_wrap.c|1059| error: (Each undeclared identifier is
> reported only once
>
> every time SWIG_ConvertPtr ist called. Which is no wonder since that
> macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
> TSRMLS_DC).
>
> No idea how to fix this :(
>
I'm afraid that the PHP module is under neglect as the PHP maintainer
isn't around anymore. I'll commit any patches until a new volunteer
comes along though.

William

[Torsten Landschoff (t-landschoff)]

Hi William,

On Tue, Feb 22, 2005 at 09:00:52PM +0000, William S Fulton wrote:
> >every time SWIG_ConvertPtr ist called. Which is no wonder since that
> >macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
> >TSRMLS_DC).
> >
> >No idea how to fix this :(
> >
> I'm afraid that the PHP module is under neglect as the PHP maintainer
> isn't around anymore. I'll commit any patches until a new volunteer
> comes along though.

Yep, that was my impression already. That also the reason why I looked
that deep into it and I'll try to get those problems fixed but I had
other things to do when I wrote that email so I thought I'd better get
that information I have out first and care about anything else later.

Greetings

 Torsten

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Tue, 22 Feb 2005 23:52:23 +0100
From: <email address hidden> (Torsten Landschoff)
To: William S Fulton <email address hidden>
Cc: Torsten Landschoff <email address hidden>,
 <email address hidden>, <email address hidden>
Subject: Re: [Swig-dev] [PATCH] Bug in PHP4 wrapper generation

Hi William,

On Tue, Feb 22, 2005 at 09:00:52PM +0000, William S Fulton wrote:
> >every time SWIG_ConvertPtr ist called. Which is no wonder since that
> >macro uses TSRMLS_CC without a definition (which would be TSRMLS_D or
> >TSRMLS_DC).
> >
> >No idea how to fix this :(
> >
> I'm afraid that the PHP module is under neglect as the PHP maintainer
> isn't around anymore. I'll commit any patches until a new volunteer
> comes along though.

Yep, that was my impression already. That also the reason why I looked
that deep into it and I'll try to get those problems fixed but I had
other things to do when I wrote that email so I thought I'd better get
that information I have out first and care about anything else later.

Greetings

 Torsten

[Matt Zimmerman (mdz)]

Seems to be triggered by an issue in a version of php4 which entered Debian post-UVF

[Torsten Landschoff (t-landschoff)]

tags 294499 - patch

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Mon, 28 Feb 2005 12:56:42 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: tagging 294499

tags 294499 - patch

[Steve Langasek (vorlon)]

Hi Torsten,

I'm happy to hack on this bug to get it fixed, but I'm afraid I'm starting
from square-one as far as knowledge of swig is concerned. Could you send me
a copy of your "simple example" that I could start from?

Thanks,
--
Steve Langasek
postmodern programmer

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Thu, 3 Mar 2005 20:49:41 -0800
From: Steve Langasek <email address hidden>
To: <email address hidden>
Subject: Re: php4 wrappers broken for thread-safe php

--i7F3eY7HS/tUJxUd
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Torsten,

I'm happy to hack on this bug to get it fixed, but I'm afraid I'm starting
=66rom square-one as far as knowledge of swig is concerned. Could you send=
 me
a copy of your "simple example" that I could start from?

Thanks,
--=20
Steve Langasek
postmodern programmer

--i7F3eY7HS/tUJxUd
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFCJ+jhKN6ufymYLloRAnr2AJ9aPVLDl5+I/BnioVJDPU7+7VB6nACgnzfO
bESLkuXETLJJGjN6bssRefw=
=DUu7
-----END PGP SIGNATURE-----

--i7F3eY7HS/tUJxUd--

[Torsten Landschoff (torsten)]

Hi Steve,

On Thu, Mar 03, 2005 at 08:49:41PM -0800, Steve Langasek wrote:
> I'm happy to hack on this bug to get it fixed, but I'm afraid I'm starting
> from square-one as far as knowledge of swig is concerned. Could you send me
> a copy of your "simple example" that I could start from?

Not sure if a simple example will cut it ;) What I used for testing this
problem was a wrapper for libcdaudio I originally started to grab my
CDs. Real world was in the way so that project did not go anywhere but
I should still build as far as I got.

I attached the sources. If you get that to build for PHP we should be
all set. There is a test.py which worked using the python bindings so
that would be a nice test after converting to PHP also. Guess it should
work if it builds though...

Thanks and Greetings

 Torsten

[Torsten Landschoff (torsten)]

Hi again,

On Fri, Mar 04, 2005 at 09:50:25AM +0100, Torsten Landschoff wrote:
> On Thu, Mar 03, 2005 at 08:49:41PM -0800, Steve Langasek wrote:
> > I'm happy to hack on this bug to get it fixed, but I'm afraid I'm starting
> > from square-one as far as knowledge of swig is concerned. Could you send me
> > a copy of your "simple example" that I could start from?

BTW: I digged a bit deeper and it is all but simple. The Zend API looks
a bit awkward to me.

I looked into currenct PHP CVS and found the soap extension to be a good
example a of module providing classes to PHP. It uses some defines new
to the PHP5 header files (PHP_METHOD etc.) which are conditionally
defined for older zend engines as well. I think this is what the wrapper
code should use as well.

This does not help in my example as the tsrm_ls (some lock pointer it
seems) is not passed to the property getters and setters so I wonder how
this can be fixed there.

Greetings

 Torsten

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Fri, 4 Mar 2005 09:50:25 +0100
From: Torsten Landschoff <email address hidden>
To: Steve Langasek <email address hidden>, <email address hidden>
Subject: Re: Bug#294499: php4 wrappers broken for thread-safe php

--5G06lTa6Jq83wMTw
Content-Type: multipart/mixed; boundary="Bn2rw/3z4jIqBvZU"
Content-Disposition: inline

--Bn2rw/3z4jIqBvZU
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Steve,=20

On Thu, Mar 03, 2005 at 08:49:41PM -0800, Steve Langasek wrote:
> I'm happy to hack on this bug to get it fixed, but I'm afraid I'm starting
> from square-one as far as knowledge of swig is concerned. Could you send=
 me
> a copy of your "simple example" that I could start from?

Not sure if a simple example will cut it ;) What I used for testing this
problem was a wrapper for libcdaudio I originally started to grab my
CDs. Real world was in the way so that project did not go anywhere but
I should still build as far as I got.

I attached the sources. If you get that to build for PHP we should be
all set. There is a test.py which worked using the python bindings so=20
that would be a nice test after converting to PHP also. Guess it should
work if it builds though...

Thanks and Greetings

 Torsten

--Bn2rw/3z4jIqBvZU
Content-Type: application/octet-stream
Content-Disposition: attachment; filename="cdaudio.tar.gz"
Content-Transfer-Encoding: base64

H4sIAB0hKEIAA+0baXPbxlVfhV+xsZMOyfDW1VhxEkWSbU1lyRPJTSeZFAGBJbkViEWAhWjW
zn/ve7u4DwJSaGfacmcECLvv2nfs+WhaRmAx3vOXbDbY+ThlOByOjw4O8D06Ohim31HZGQ3H
o/FwPNzbP9wZjvYP9/d2yMFHkidTAl8YHiE7gnu+oE4lXF171JHo/V9SzLT9w4++v5xtkgfq
43B/v8L+YOzhHth/dLh/eHAwPgS40dFwPN4hn0SJ/+f2/2LBrcCmJDS9pn3xXnvKHNMOLEq+
jhxi/o32xe/QFjU8ESuXLgzX77MnmuYLLzAFsZhv6syZcvJeI4Q5YY3rUZ864pisK4MOufAl
PAnhvyWdAZChTrAA4fRQEh2sJQJfD2mD8PQ4S+Y08Dwa8ibYruikZRRsQe8NO/zwDPNOVh2X
0pHtBNvrCGVJlAvUgI5NnZmYHxfo3HIRghAFoujEejYVH9WhKm0XOpalMWXg5OspSBovEC6k
wB0i5lRba97yIvuSFUBgJ5UAfqUEIMBVsJhQj/CpEsIHKf6YBKE5lDdIJ0684+fXJ//Qb384
Of3bzS/HkQS3svO+S002ZeZjmCMXb2EIBhoEGX4/1rQC3TxQUcz3Fb6kQMqcCZhcKgeK1Fft
kYqKy4vGACpvuM+kYFk6LHIu3Z4Y68M+lIbPmAnMvre5efcYTZ5YFgwafnbECAMbBio9kgc/
ioF1enby9uziWllYl/8/RgjuPQYry/zs5PYk8gWNvhPUc4jNnVk0POszKu6p54PSW/ecWW3l
My8Cx5SGcD0OEcRdKnWBEKiJCMOcw0zXmQTTKfW60kzgHkgC/wW4qWcsKIytHA3XwkpVAyAh
BNRjs6qWEFCTNC/8aaq51J/gXYBHbmuAo24k4gFeKUIH/1E9iyWPsEPRH4IaickcJnSL3jOT
hjpUH7pjpHszBTh/3gq/LOqbSRtOW+mWLilMmp3rt7dv3t4mOC637WocNRGSjnonWIFrGYK2
iuTx31ISMYWYsW2s0jaO+eMHLpeEbFTf1LHkV0puQDeFXUTlLjqpn6KjgrRL+v1+Fl+PB54K
CRSitrurwiiUJCQXQwF+Gd21NDO0ctil8tQyrUBJaVz11rDuDQd8rAAe9yzqb31oldKqR/Nh
8Cj3YNcIfFreBGNvsKhoo/+ipihvMm1eoBjLQYV+z+0c2WwPVDuBV0Jz1hyvoxC1cHy1Jjq2
MqtUWGiFsdWESUYPPDuKLlk/57AU6uCzS0xwb6iWYwTApdE9amC3PBiMbeaLDAlAm5IOPtHC
uyTdlqCQDj6rIEgHBHy3SrNcekzQtTwTllU8E5ZlPCOWqseyZUYdj7aSZXtUBZOwfCdGlhr3
wD30hakbNkz+mVELKmEkM2DggmcqbPx0F2PMSmtnyCTzgcKdepRWM80xMrmLwyJfSJByNB4I
Ka7SWEYQBcCcPGX8TtFeZLVQS3ZRoJyQBvM6GH5l3qLeaUFC6BCi6OPSxVGCSucrtM4pGIfI
Z3aQR8w75upzIeABoQEM5QDKU2NuFDWC31Enbg5dAet+3vslDfxbQL2SwVYhxR6pegkTkdw9
glviCyQPF2Rp8SVB0pGvgvwo2h9kluYFWzLgJZ/dEr9RQZAT4beAiVK1wbo8nLsf3/nCwBNy
zxlHxoLcvxVCrzySYBSgHixR9MC5c/jSUT2vD...

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Fri, 4 Mar 2005 10:25:56 +0100
From: Torsten Landschoff <email address hidden>
To: Torsten Landschoff <email address hidden>
Cc: Steve Langasek <email address hidden>, <email address hidden>
Subject: Re: Bug#294499: php4 wrappers broken for thread-safe php

--IU5/I01NYhRvwH70
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi again,=20

On Fri, Mar 04, 2005 at 09:50:25AM +0100, Torsten Landschoff wrote:
> On Thu, Mar 03, 2005 at 08:49:41PM -0800, Steve Langasek wrote:
> > I'm happy to hack on this bug to get it fixed, but I'm afraid I'm start=
ing
> > from square-one as far as knowledge of swig is concerned. Could you se=
nd me
> > a copy of your "simple example" that I could start from?

BTW: I digged a bit deeper and it is all but simple. The Zend API looks
a bit awkward to me.=20

I looked into currenct PHP CVS and found the soap extension to be a good
example a of module providing classes to PHP. It uses some defines new
to the PHP5 header files (PHP_METHOD etc.) which are conditionally
defined for older zend engines as well. I think this is what the wrapper
code should use as well.

This does not help in my example as the tsrm_ls (some lock pointer it
seems) is not passed to the property getters and setters so I wonder how=20
this can be fixed there.=20

Greetings

 Torsten

--IU5/I01NYhRvwH70
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFCKCmkdQgHtVUb5EcRAhCGAJ0bsMR5KVBGCodq43hW1Lm0Ux4EGACfQV9I
ecKRgcNyNUyI8YtOL1EBrbg=
=L5I1
-----END PGP SIGNATURE-----

--IU5/I01NYhRvwH70--

[Steve Langasek (vorlon)]

Hi Torsten,

It seems that because of this and several other bugs that are not easily
resolvable in the sarge time frame, as well as pressure from upstream, we
are going to be backing out PHP's ZTS support in the next day or so; at
which point this bug will be a non-issue for us.

Cheers,
--
Steve Langasek
postmodern programmer

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Wed, 23 Mar 2005 00:47:28 -0800
From: Steve Langasek <email address hidden>
To: <email address hidden>
Subject: Re php4 wrappers broken for thread-safe php

--wULyF7TL5taEdwHz
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Torsten,

It seems that because of this and several other bugs that are not easily
resolvable in the sarge time frame, as well as pressure from upstream, we
are going to be backing out PHP's ZTS support in the next day or so; at
which point this bug will be a non-issue for us.

Cheers,
--=20
Steve Langasek
postmodern programmer

--wULyF7TL5taEdwHz
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFCQS0eKN6ufymYLloRAsxRAJ9dgOUXvTihlvGj0bvrz/CAUg4M7ACdGHwa
E+VRVKmAAOfVsAGw3VuT6Es=
=u4Sp
-----END PGP SIGNATURE-----

--wULyF7TL5taEdwHz--

[Torsten Landschoff (torsten)]

Hi Steve,

On Wed, Mar 23, 2005 at 12:47:28AM -0800, Steve Langasek wrote:
> It seems that because of this and several other bugs that are not easily
> resolvable in the sarge time frame, as well as pressure from upstream, we
> are going to be backing out PHP's ZTS support in the next day or so; at
> which point this bug will be a non-issue for us.

Thanks for the info. The bug remains valid though as swig should be able
to cope with ZTS. But I'll lower the severity to normal.

Greetings

 Torsten

[Torsten Landschoff (t-landschoff)]

severity 294499 normal

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Wed, 23 Mar 2005 11:24:40 +0100
From: Torsten Landschoff <email address hidden>
To: Steve Langasek <email address hidden>, <email address hidden>
Subject: Re: Bug#294499: Re php4 wrappers broken for thread-safe php

--cWoXeonUoKmBZSoM
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Steve,=20

On Wed, Mar 23, 2005 at 12:47:28AM -0800, Steve Langasek wrote:
> It seems that because of this and several other bugs that are not easily
> resolvable in the sarge time frame, as well as pressure from upstream, we
> are going to be backing out PHP's ZTS support in the next day or so; at
> which point this bug will be a non-issue for us.

Thanks for the info. The bug remains valid though as swig should be able
to cope with ZTS. But I'll lower the severity to normal.

Greetings

 Torsten

--cWoXeonUoKmBZSoM
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFCQUPodQgHtVUb5EcRArVeAJ4nshjWoxiUgUPU6gX8KKqRNwJfrgCffyW5
p31OYyNI6QHeENUhyotF2Ls=
=VzcX
-----END PGP SIGNATURE-----

--cWoXeonUoKmBZSoM--

[Debian Bug Importer (debzilla)]

Message-Id: <email address hidden>
Date: Wed, 23 Mar 2005 11:24:50 +0100 (CET)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: severity of 294499 is normal

severity 294499 normal

[wsfulton (wsf)]

SWIG-1.3.27 contains the swig-fix-for-threadsafe-ZTS.patch patch.
Assumed fixed and closing corresponding swig bugtracker bug
http://sourceforge.net/tracker/index.php?func=detail&aid=653355&group_id=1645&atid=101645

William

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Sat, 05 Nov 2005 23:13:02 +0000
From: William S Fulton <email address hidden>
To: <email address hidden>
Subject: Bug fixed in SWIG-1.3.27

SWIG-1.3.27 contains the swig-fix-for-threadsafe-ZTS.patch patch.
Assumed fixed and closing corresponding swig bugtracker bug
http://sourceforge.net/tracker/index.php?func=detail&aid=653355&group_id=1645&atid=101645

William

[Torsten Landschoff (torsten)]

Hi William,

On Sat, Nov 05, 2005 at 11:13:02PM +0000, William S Fulton wrote:
> SWIG-1.3.27 contains the swig-fix-for-threadsafe-ZTS.patch patch.
> Assumed fixed and closing corresponding swig bugtracker bug
> http://sourceforge.net/tracker/index.php?func=detail&aid=653355&group_id=1645&atid=101645

Thanks, that's great to know. I'll update the Debian package.

Greetings

 Torsten

[Debian Bug Importer (debzilla)]

Message-ID: <email address hidden>
Date: Sun, 6 Nov 2005 15:42:35 +0100
From: Torsten Landschoff <email address hidden>
To: William S Fulton <email address hidden>,
 <email address hidden>
Subject: Re: Bug#294499: Bug fixed in SWIG-1.3.27

--bp/iNruPH9dso1Pn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi William,=20

On Sat, Nov 05, 2005 at 11:13:02PM +0000, William S Fulton wrote:
> SWIG-1.3.27 contains the swig-fix-for-threadsafe-ZTS.patch patch.=20
> Assumed fixed and closing corresponding swig bugtracker bug
> http://sourceforge.net/tracker/index.php?func=3Ddetail&aid=3D653355&group=
_id=3D1645&atid=3D101645

Thanks, that's great to know. I'll update the Debian package.

Greetings

 Torsten

--bp/iNruPH9dso1Pn
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

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

iD8DBQFDbhZbdQgHtVUb5EcRAvmyAJ0YvX7htrmK5V3QdNvcKTQU5DqEkgCfdgdL
eGHbN0X1DcejZJ6nDkTsDLI=
=k0Em
-----END PGP SIGNATURE-----

--bp/iNruPH9dso1Pn--