After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 547781 - Embedding perl needs PERL_SYS_INIT3()
Embedding perl needs PERL_SYS_INIT3()
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: General
1.8.x
Other Linux
: Normal major
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2008-08-14 14:53 UTC by J.H.M. Dassen (Ray)
Modified: 2008-08-19 13:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (1.10 KB, patch)
2008-08-14 14:55 UTC, J.H.M. Dassen (Ray)
accepted-commit_now Details | Review

Description J.H.M. Dassen (Ray) 2008-08-14 14:53:29 UTC
Originally reported as http://bugs.debian.org/495062:
----------------------------------------------------------------------------
Package: gnumeric
Version: 1.8.3-4
Severity: important
User: debian-perl@lists.debian.org
Usertags: perl-sys-init3

As described in the 'perlembed' document, programs embedding Perl
must use the PERL_SYS_INIT3() and PERL_SYS_TERM() macros to provide
system-specific tune up of the C runtime environment necessary to run
Perl interpreters.

Your package has been identified as failing this:

- at least one of the binary packages built from the source depends
on libperl5.10

- the unpacked source matches 'perl_parse' but not 'PERL_SYS_INIT3'

As a consequence, the embedded Perl interpreter is most probably
broken on the hppa architecture, where PERL_SYS_INIT3() is needed for
initializing lock structures. Without this, calling perl_parse() will
hang inside pthread_mutex_lock(). See #486069 for more information.

Cheers, 
-- 
Niko Tyni   ntyni@debian.org
----------------------------------------------------------------------------
Comment 1 J.H.M. Dassen (Ray) 2008-08-14 14:55:21 UTC
Created attachment 116581 [details] [review]
Proposed patch
Comment 2 Jody Goldberg 2008-08-19 11:24:16 UTC
Comment on attachment 116581 [details] [review]
Proposed patch

The docs are clear.  My only worry would be the implications of a user enabling then disabling and enabling again.  We lack a 'can not disable this till you exit' capability.

Please commit both branches.