[Bug 215610] New: let's build php with system pcre, if possible
https://bugzilla.novell.com/show_bug.cgi?id=215610 Summary: let's build php with system pcre, if possible Product: openSUSE 10.2 Version: Beta 1 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: Development AssignedTo: anosek@novell.com ReportedBy: poeml@novell.com QAContact: qa@suse.de At the moment, php is built with its own private copy of pcre. This can mean duplication of work in case of a pcre security update, or missing to fix pcre at all, or clashes if to copies of pcre are loaded into apache (apache links against system pcre). Using system pcre wasn't possible in the past, it was tried, and led to problems because apache used its own pcre as well. But this is fixed now. History of related bugs:
Bug 112645 - build php4 and php5 with system pcre Bug 114157 - VUL-0: after last pcre and php update some php pages now crash Bug 112651 - VUL-0: apache2 - fix pcre problems
there was a problem with older versions when apache was _also_ linking to its own pcre. This has been corrected in 10.1.
Since it uses system pcre, there shouldn't be a problem any more, should there?
But, citing Cristian:
hrmm.. that will need a quick look , and a system PCRE >= 6.7 the look is needed to see if the PCRE library bundled with php is not modified.,.. at least sqlite , sqlite3, gd and zip bundled libraries are .
and the old adagio is still valid.. " if aint' broke.. don't break it... " ;P , anyway, I 'll attempt to break it if you like later ;)
Ales, maybe we could just give it a try? (If pcre is not modified in a way which prevents the switch) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=215610 judas_iscariote@shorewall.net changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED ------- Comment #1 from judas_iscariote@shorewall.net 2006-11-05 03:22 MST ------- fixed in buildservice, not tested at all though..pleas get PHP5 packages from server:php QA of this in combiation with mod_rewrite is really needed and **ASAP**. thanks for your attention. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=215610 ------- Comment #2 from poeml@novell.com 2006-11-05 08:27 MST ------- Hm, too bad that it doesn't work with 10.1's pcre (too old). This makes it somewhat harder to test... BTW, I'm replacing the check for suse_version > 1001 with suse_version 1010, which is the correct one :) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=215610 ------- Comment #3 from judas_iscariote@shorewall.net 2006-11-06 01:32 MST ------- (In reply to comment #2)
Hm, too bad that it doesn't work with 10.1's pcre (too old). This makes it somewhat harder to test...
I tested it quikcly with drupal mod_rewrite usage and seems to work properly :-)
BTW, I'm replacing the check for suse_version > 1001 with suse_version 1010, which is the correct one :)
Thank you Peter, my mistake, that incorrect number produced an strange result I was wondering why :-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
https://bugzilla.novell.com/show_bug.cgi?id=215610 mmarek@novell.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mmarek@novell.com Status|ASSIGNED |RESOLVED Resolution| |FIXED ------- Comment #4 from mmarek@novell.com 2006-11-06 02:40 MST ------- Seems OK then (I also checked the bundled pcre lib the other day and it didn't differ from the system one), submitted to factory. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.
participants (1)
-
bugzilla_noreply@novell.com