Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.0.0

Request #15895 RedHat Support
Submitted: 2009-02-12 23:35 UTC
From: ifeghali Assigned: kvz
Status: Closed Package: System_Daemon
PHP Version: 5.1.6 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2009-02-12 23:35 UTC] ifeghali (Igor Feghali)
Description: ------------ Patch attached. Test script: --------------- Expected result: ---------------- Actual result: --------------

Comments

 [2009-02-12 23:41 UTC] ifeghali (Igor Feghali)
Still can't attach patches... Please wait while fixed.
 [2009-02-13 21:12 UTC] ifeghali (Igor Feghali)
The following patch has been added/updated: Patch Name: patch_to_redhat.diff URL: patch patch_to_redhat.diff
 [2009-02-13 21:19 UTC] ifeghali (Igor Feghali)
The following patch has been added/updated: Patch Name: template_sysvinit.diff URL: patch template_sysvinit.diff
 [2009-02-15 17:30 UTC] kvz (Kevin van Zonneveld)
-Assigned To: +Assigned To: kvz
 [2009-02-15 17:55 UTC] kvz (Kevin van Zonneveld)
Thanks a lot for your feature request. And thanks even more for supplying patches Igor! I've made some changes to your patches cause I like to keep my OS classes light. Basically: providing variables in the OS classes should be enough to for the base OS class to work magic. The isInstalled already works if the osVersionFile has been found (which not only enabled CentOS, Redhat but also Unbreakable, fedora & whitebox support. If more specifics are needed, we can easily let class System_Daemon_OS_CentOS extends System_Daemon_OS_Redhat). And I made a '#datadir#' expression that's substituted by the actual datadir. Please let me know if it works. Since CentOS is like 4 ISOs big, it's hard for me to setup a virtualbox & test myself.
 [2009-02-15 18:05 UTC] kvz (Kevin van Zonneveld)
-Status: Assigned +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/
 [2009-02-15 18:42 UTC] ifeghali (Igor Feghali)
Thank you for committing the patches Kevin. I will test it on Monday and will let you know if it works.
 [2009-02-16 18:39 UTC] ifeghali (Igor Feghali)
Hello Kevin, the RH template is working gracefully. Regards, Igor Feghali.
 [2009-02-17 14:24 UTC] kvz (Kevin van Zonneveld)
Ah, nice to hear! Great that we have RedHat support now as well. Thanks to you Igor!