Tickets :: [#7032] allow the administrator to set the PEAR directory

6.0.0-beta1

7/8/25

Summary	allow the administrator to set the PEAR directory
Queue	Horde Base
Queue Version	3.2.1
Type	Enhancement
State	Rejected
Priority	1. Low
Owners
Requester	develop (at) kristov (dot) de
Created	07/06/2008 (6211 days ago)
Due
Updated	07/07/2008 (6210 days ago)
Assigned
Resolved	07/06/2008 (6211 days ago)
Milestone
Patch	Yes

07/07/2008 09:09:02 AM	Jan Schneider	Comment #4	Reply to this comment
That's sad, I know, but it would probably speed things up if you provide a patch for that issue. I'd rather not want to hack around deficiencies in external packages, especially if they are still beta code, due to change.

07/07/2008 07:03:02 AM	develop (at) kristov (dot) de	Comment #3	Reply to this comment
You're right, of course. But for affected Horde users this patch (combined with a corresponding holidays.php patch which I have written but not submitted yet) is currently the only way to make displaying holidays work. The PEAR bug you provided the link for has not received much attention for more than a year...

07/06/2008 10:15:20 PM	Jan Schneider	Comment #2 State ⇒ Rejected	Reply to this comment
There is only one single place where we use PEAR_INSTALL_DIR, in holiday.php. And this is only due to the incomplete design of Date_Holidays, see http://pear.php.net/bugs/bug.php?id=10221

07/06/2008 07:27:06 PM	develop (at) kristov (dot) de	Comment #1 Priority ⇒ 1. Low New Attachment: horde-3.2.1-pear.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ Horde Base Summary ⇒ allow the administrator to set the PEAR directory Type ⇒ Enhancement State ⇒ New	Reply to this comment
There are some PHP installations in the wild where PEAR_INSTALL_DIR is empty (cf. http://pear.php.net/bugs/bug.php?id=4597). However, some Horde components do rely on it being properly set (kronolith/lib/Driver/holidays.php for example). To allow to have this work, it is probably advisable to let the user configure the location of the PEAR libraries. The patch attached addresses this issue and adds a new configuration variable 'peardir'.