Opened 18 years ago

Closed 17 years ago

Last modified 15 years ago

#8289 closed enhancement (fixed)

[PATCH] php5 with +apache2 needs explicit link with DP pcre

Reported by: blair (Blair Zajac) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version: 1.2
Keywords: Cc: ryandesign (Ryan Carsten Schmidt)
Port:

Description

Building apache2 and php5 today on an older 10.3 box, putting this line into httpd.conf

LoadModule php5_module modules/libphp5.so

got Apache to quickly core dump. Here's the stack trace:

gdb httpd /cores/core.7502 GNU gdb 5.3-20030128 (Apple version gdb-330.1) (Fri Jul 16 21:42:28 GMT 2004) Copyright 2003 Free Software Foundation, Inc. GDB is free software, covered by the GNU General Public License, and you are welcome to change it and/or distribute copies of it under certain conditions. Type "show copying" to see the conditions. There is absolutely no warranty for GDB. Type "show warranty" for details. This GDB was configured as "powerpc-apple-darwin". Reading symbols for shared libraries ....... done Core was generated by `/Volumes/MYDRIVEMAC/MyDrive/servers/mac_ppc/apache2/bin/httpd'. #0 0x8fe01400 in dyld_halt () (gdb) bt #0 0x8fe01400 in dyld_halt () #1 0x8fe0a2b0 in dyld_check_and_report_undefineds () #2 0x8fe0489c in dyld_unload_bundle_image () #3 0x8fe13230 in dylddyld_unlink_module () #4 0x9003db4c in NSUnLinkModule () #5 0x00314af4 in ?? () #6 0x0031a53c in ?? () #7 0x00319ba0 in ?? () #8 0x000030cc in sha_transform () at pcre_compile.c:1956

It looks like PHP 5.1.2 comes with pcre 6.2 and DP has 6.6 now. And Apache2 links against the DP pcre 6.6. So not too surprising there's a core dump.

This patch adds pcre explicitly to only the apache2 variant, but it may be a good idea to add it to the entire package. That's up to the maintainer.

Regards, Blair

PS There's no explicit maintainer on PHP5, although it's seen recent activity.

Attachments (1)

dp-php5-with-apache2-core-dump.txt (689 bytes) - added by blair (Blair Zajac) 18 years ago.
Patch to add explicit port:pcre and link against DP's pcre port

Download all attachments as: .zip

Change History (7)

Changed 18 years ago by blair (Blair Zajac)

Patch to add explicit port:pcre and link against DP's pcre port

comment:1 Changed 18 years ago by opendarwin-2006@…

Owner: changed from darwinports-bugs@… to opendarwin-2006@…

Assign to maintainer

comment:2 Changed 18 years ago by markd@…

Type: defectenhancement

comment:3 Changed 17 years ago by pipping@…

Milestone: Port Bugs

comment:4 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Owner: changed from opendarwin-2006@… to ryandesign@…

comment:5 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Although I am not experiencing the issue mentioned in this ticket, it seems reasonable that php5 should link against the pcre port. I am in favor of that happening always, not just for the apache2 version. Changed in r23411.

comment:6 Changed 15 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.