swig: PHP generation is broken with -zts enabled (and it is now the default in sid)

Bug #13512 reported by Debian Bug Importer
4
Affects Status Importance Assigned to Milestone
swig1.3 (Debian)
Fix Released
Unknown
swig1.3 (Ubuntu)
Invalid
High
Unassigned

Bug Description

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

Revision history for this message
Debian Bug Importer (debzilla) wrote :

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

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Fri, 04 Mar 2005 08:46:41 +0100
From: Pierre Habouzit <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: swig: PHP generation is broken with -zts enabled (and it is now the default
 in sid)

Package: swig
Severity: grave
Tags: sid
Justification: renders package unusable

  with zts enabled, any PHP generation generates a lot of errors. in
fact, any PHP bindings generated by SWIG now FTBFS with such errors :

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

  the problem is know upstreams (like said here [1]), but they don't
seem very hasty in fixing it.

  Since this break all the packages generated by SWIG in debian, this is
at least grave, and should really be fixed. (I'm sorry, I've not enough
php internals knowledge to help you on that).

  you can read #294003 [2] that is an example of php binding generated
by swig that is now broken.

  [1] http://mailman.cs.uchicago.edu/pipermail/swig-dev/2005-February/014744.html

  [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=294003

Revision history for this message
In , Torsten Landschoff (torsten) wrote : Re: Bug#298028: swig: PHP generation is broken with -zts enabled (and it is now the default in sid)

Hi Pierre,

On Fri, Mar 04, 2005 at 08:46:41AM +0100, Pierre Habouzit wrote:
> Package: swig
> Severity: grave
> Tags: sid
> Justification: renders package unusable

Come on, you must be kidding.

1) the problem is already known and you are even referencing that other
   bug report
2
) this breaks only php wrappers generated by swig. Given that swig can
   generate wrappers for 15 other languages I'd hardly agree that this
   makes the package unusable. For example the subversion bindings for
   perl and python are generated using this swig and are working just
   fine.

> with zts enabled, any PHP generation generates a lot of errors. in
> fact, any PHP bindings generated by SWIG now FTBFS with such errors :
>
> foo.c|####| error: `tsrm_ls' undeclared (first use in this function) ...
> foo.c|####| error: (Each undeclared identifier is reported only once

I know.

> the problem is know upstreams (like said here [1]), but they don't

Yes, I reported it.

> seem very hasty in fixing it.

No, they don't have the knowledge of PHP.

> Since this break all the packages generated by SWIG in debian, this is

This is false.

> at least grave, and should really be fixed. (I'm sorry, I've not enough

So this is false as well.

> php internals knowledge to help you on that).

Me neither.

> you can read #294003 [2] that is an example of php binding generated
> by swig that is now broken.

That's interesting, I did not know of any package using swig to generate
PHP bindings. I'll try to find a way to fix this. Steve Langasek already
offered his help as well.

Greetings

 Torsten

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Fri, 4 Mar 2005 09:56:38 +0100
From: Torsten Landschoff <email address hidden>
To: Pierre Habouzit <email address hidden>, <email address hidden>
Subject: Re: Bug#298028: swig: PHP generation is broken with -zts enabled (and it is now the default
 in sid)

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

Hi Pierre,=20

On Fri, Mar 04, 2005 at 08:46:41AM +0100, Pierre Habouzit wrote:
> Package: swig
> Severity: grave
> Tags: sid
> Justification: renders package unusable

Come on, you must be kidding.

1) the problem is already known and you are even referencing that other
   bug report
2
) this breaks only php wrappers generated by swig. Given that swig can
   generate wrappers for 15 other languages I'd hardly agree that this=20
   makes the package unusable. For example the subversion bindings for
   perl and python are generated using this swig and are working just
   fine.

> with zts enabled, any PHP generation generates a lot of errors. in
> fact, any PHP bindings generated by SWIG now FTBFS with such errors :
>=20
> foo.c|####| error: `tsrm_ls' undeclared (first use in this function) .=
=2E.
> foo.c|####| error: (Each undeclared identifier is reported only once

I know.

> the problem is know upstreams (like said here [1]), but they don't

Yes, I reported it.

> seem very hasty in fixing it.

No, they don't have the knowledge of PHP.

> Since this break all the packages generated by SWIG in debian, this is

This is false.=20

> at least grave, and should really be fixed. (I'm sorry, I've not enough

So this is false as well.

> php internals knowledge to help you on that).

Me neither.=20

> you can read #294003 [2] that is an example of php binding generated
> by swig that is now broken.

That's interesting, I did not know of any package using swig to generate
PHP bindings. I'll try to find a way to fix this. Steve Langasek already=20
offered his help as well.

Greetings

 Torsten

--ZJcv+A0YCCLh2VIg
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)

iD8DBQFCKCLGdQgHtVUb5EcRAhlnAJ9XRpCXFaVVceLKBSMmaasfH6gZTACdELcs
A4+bUIOffUlU9BoEKAZD/To=
=YqeU
-----END PGP SIGNATURE-----

--ZJcv+A0YCCLh2VIg--

Revision history for this message
Matt Zimmerman (mdz) wrote :

We avoided the -zts breakage

Revision history for this message
In , John Lenz (lenz) wrote :

I would just add a warning to SWIG, so that if it is run with the -php
argument it outputs a warning like "PHP wrapper support is currently broken
with Debian. Please see Sourceforge bug whatever (provide a link maybe
too)".

I would also say since it is only a problem with threading if a user really
needed to use SWIG on something, they could recompile apache and recompile
php themselves and then still use the SWIG that would be in debian, just
ignoring the error.

Thus this SWIG package is usable, even from a php standpoint. Lastly, not
very many people use SWIG for php... php doesn't even support multiple
modules sharing type information (%import).

John

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-id: <1110422768l.15808l.2l@hydra>
Date: Thu, 10 Mar 2005 02:46:08 +0000
From: John Lenz <email address hidden>
To: Torsten Landschoff <email address hidden>,
 Pierre Habouzit <email address hidden>, <email address hidden>
Subject: Re: Bug#298028: swig: PHP generation is broken with -zts enabled (and
 it is now the default in sid)

I would just add a warning to SWIG, so that if it is run with the -php
argument it outputs a warning like "PHP wrapper support is currently broken
with Debian. Please see Sourceforge bug whatever (provide a link maybe
too)".

I would also say since it is only a problem with threading if a user really
needed to use SWIG on something, they could recompile apache and recompile
php themselves and then still use the SWIG that would be in debian, just
ignoring the error.

Thus this SWIG package is usable, even from a php standpoint. Lastly, not
very many people use SWIG for php... php doesn't even support multiple
modules sharing type information (%import).

John

Revision history for this message
In , Steve Langasek (vorlon) wrote : severity of 298028 is minor

# Automatically generated email from bts, devscripts version 2.8.14
severity 298028 minor

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Sun, 27 Mar 2005 15:26:25 -0800
From: Steve Langasek <email address hidden>
To: <email address hidden>
Subject: severity of 298028 is minor

# Automatically generated email from bts, devscripts version 2.8.14
severity 298028 minor

Revision history for this message
In , Pierre Habouzit (madcoder) wrote : I close the bug since -zts is not enabled anymore

Thanks.

Revision history for this message
In , Torsten Landschoff (t-landschoff) wrote : reopening 298028

reopen 298028

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Thu, 19 May 2005 09:45:45 +0200 (CEST)
From: <email address hidden> (Pierre Habouzit)
To: <email address hidden>
Subject: I close the bug since -zts is not enabled anymore

Thanks.

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-Id: <email address hidden>
Date: Thu, 19 May 2005 10:12:19 +0200 (CEST)
From: <email address hidden> (Torsten Landschoff)
To: <email address hidden>
Subject: reopening 298028

reopen 298028

Revision history for this message
In , Torsten Landschoff (torsten) wrote : PHP should work with ZTS again

For all I know this problem was fixed upstream. I don't have any php
with ZTS enabled here now but I assume the reports are right and
therefore close this bug in the hopes that it will be reported again
should the problem show up.

Greetings

 Torsten

Revision history for this message
Debian Bug Importer (debzilla) wrote :

Message-ID: <email address hidden>
Date: Sat, 3 Dec 2005 16:23:50 +0100
From: Torsten Landschoff <email address hidden>
To: <email address hidden>
Subject: PHP should work with ZTS again

--CE+1k2dSO48ffgeK
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

For all I know this problem was fixed upstream. I don't have any php
with ZTS enabled here now but I assume the reports are right and
therefore close this bug in the hopes that it will be reported again
should the problem show up.

Greetings

 Torsten

--CE+1k2dSO48ffgeK
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)

iD8DBQFDkbiGdQgHtVUb5EcRAhfwAJ9ZtOH+smYu14ZWfBtINJZnq4GDpACfVuGu
ARzkciTgyY1Eb3MwGFYYkbU=
=FBjw
-----END PGP SIGNATURE-----

--CE+1k2dSO48ffgeK--

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.