PacketFence - BTS - PacketFence
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000991PacketFenceupstreampublic2010-05-25 11:132011-12-30 23:49
Reporterobilodeau 
Assigned Tofgaudreault 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusresolvedResolutionno change required 
PlatformOSOS Version
Product Version1.8.7 
Target VersionFixed in Version 
fixed in git revision
fixed in mtn revision
Summary0000991: php-pear-Log problem
DescriptionWe received an anonymous bug report:

Error Message: : Assigning the return value of new by reference is deprecated in /usr/share/pear/Log.php on line 169.

Looks like an upstream problem caused by the deprecation of something in php.

Deprecation is Introduced in php 5.3 maybe? http://www.php.net/manual/en/language.references.php#93292 [^]

I would hope that latest pear Log doesn't have this problem. For now, our target platform is not affected so this won't be a priority fix. We will just wait for this to settle by themselves.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-05-25 11:13obilodeauNew Issue
2010-05-25 11:13obilodeauStatusnew => assigned
2010-05-25 11:13obilodeauAssigned To => obilodeau
2010-05-25 11:14obilodeauTarget Version => 1.9.1
2010-09-22 16:02obilodeauTarget Version1.9.1 => 1.9.2
2010-11-02 12:37obilodeauNote Added: 0001746
2010-11-19 14:24obilodeauTarget Version1.9.2 => 2.0.0
2011-01-18 09:47obilodeauTarget Version2.0.0 => 2.0.1
2011-01-26 15:30obilodeauTarget Version2.0.1 => 2.0.2
2011-03-03 15:19obilodeauTarget Version2.0.2 => +1
2011-03-18 16:23fgaudreaultNote Added: 0001967
2011-03-24 10:00fgaudreaultNote Added: 0001987
2011-03-24 10:03obilodeauNote Added: 0001988
2011-03-24 10:04obilodeauAssigned Toobilodeau => fgaudreault
2011-03-24 10:08fgaudreaultNote Added: 0001989
2011-03-24 10:14obilodeauNote Added: 0001990
2011-03-24 10:21fgaudreaultNote Added: 0001991
2011-09-09 16:13fgaudreaultNote Added: 0002194
2011-09-09 16:13fgaudreaultStatusassigned => resolved
2011-09-09 16:13fgaudreaultResolutionopen => no change required
2011-12-30 23:49obilodeauTarget Version+1 =>

Notes
(0001746)
obilodeau   
2010-11-02 12:37   
current debian stable is also not running 5.3.x
(0001967)
fgaudreault   
2011-03-18 16:23   
Regarding this bug : http://pear.php.net/bugs/bug.php?id=16743 [^]

 [2009-12-27 18:57 UTC] jon (Jon Parise)
-Status: Feedback +Status: Closed
Thank you for your bug report. This issue has been fixed in the latest released version of the package, which you can download at http://pear.php.net/get/ [^]

It has been filed for PEAR/Log version 1.11.5. CentOS 5.5 is using version 1.9.13. Maybe we should install the package using pear instead of the deprecated RPM?


Not tested tho.
(0001987)
fgaudreault   
2011-03-24 10:00   
- EPEL 5.5 now use php-pear-log 1.11.6, and EPEL 6 use php-pear-log 1.12.5

This bug should be resolved in those two packages.
(0001988)
obilodeau   
2011-03-24 10:03   
Our package pulls down php-pear-Log (capital L) not php-pear-log and they originate from different repos. See 0000804.

So it is not fixed upstream unless we switch to php-pear-log but can we just do that?
(0001989)
fgaudreault   
2011-03-24 10:08   
Its the same name from both repos, we just need to install EPEL and we are done:

Installed Packages
Name : php-pear-Log
Arch : noarch
Version : 1.9.13
Release : 1.el5.centos
Size : 204 k
Repo : installed
Summary : Abstracted logging facility for PHP
URL : http://pear.php.net/package/Log [^]
License : PHP
Description: The Log framework provides an abstracted logging system.
           : It supports logging to console, file, syslog, SQL, Sqlite, mail, and mcal
           : targets. It also provides a subject - observer mechanism.
           :
           : php-pear-Log can optionally use package "php-pear-DB" (version >= 1.3)
           : and "php-pear-MDB2" (version >= 2.0.0RC1).

Available Packages
Name : php-pear-Log
Arch : noarch
Version : 1.11.6
Release : 1.el5
Size : 56 k
Repo : epel
Summary : Abstracted logging facility for PHP
URL : http://pear.php.net/package/Log [^]
License : MIT
Description: The Log framework provides an abstracted logging system.
           : It supports logging to console, file, syslog, SQL, Sqlite, mail, and mcal
           : targets. It also provides a subject - observer mechanism.
           :
           : php-pear-Log can optionally use package "php-pear-DB" (version >= 1.3)
           : and "php-pear-MDB2" (version >= 2.0.0RC1).
(0001990)
obilodeau   
2011-03-24 10:14   
See our admin guide, for RHEL EPEL is mandatory but for CentOS I thought that everything was provided in CentOS-Extras?

So we would need to add EPEL to our CentOS installation process, correct?

Again is this really worth the effort? Right now as said in the bug description our target platform (CentOS / RHEL) works just fine without any changes..
(0001991)
fgaudreault   
2011-03-24 10:21   
No need to push that, unless someone would abosolutely want to use php 5.3 on a CentOS install. EPEL doesn't provide the php package. CentOS 6 will ship with php 5.3, but with proper php-pear-Log module as well.

So I guess we should just keep this bug as a reference.
(0002194)
fgaudreault   
2011-09-09 16:13   
Let's close this one.

In CentOS 5, this bug doesn't apply.
In CentOS 6, the distro ships with 5.3 and a working php-pear-Log package.