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

Bug #20469 Uses php 5.4 syntax, but pear dependency is 5.3.0
Submitted: 2014-12-18 22:05 UTC
From: glen Assigned: glen
Status: Closed Package: Net_SmartIRC (version 1.1.4)
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


 [2014-12-18 22:05 UTC] glen (Elan Ruusamäe)
Description: ------------ should go back to 5.3 syntax (only one place found erratic) or fix package depednency to require php 5.4, not 5.3.0 like it is now

Comments

 [2014-12-18 22:06 UTC] glen (Elan Ruusamäe)
 [2014-12-26 08:37 UTC] garrettw (Garrett Whitehorn)
-Status: Open +Status: Closed -Assigned To: +Assigned To: garrettw
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/
 [2014-12-29 16:19 UTC] glen (Elan Ruusamäe)
technically [glen] fixed the bug, [garrettw] only approved and committed it as i don't have access complaining about package changelog...
 [2014-12-30 15:08 UTC] garrettw (Garrett Whitehorn)
Maybe you're right. But please be aware that you do, in fact, have access to submit changes to the source code since it is hosted on Github. If you don't have an account there, make one, fork the Net_SmartIRC repo, make your changes to your copy of the code, and then submit a pull request. I'm not aware of any way to merge patches from here with the Github repo aside from manually, so that's what I had to do.
 [2014-12-30 15:10 UTC] garrettw (Garrett Whitehorn)
-Assigned To: garrettw +Assigned To: glen
 [2014-12-30 16:45 UTC] glen (Elan Ruusamäe)
ah, i see. was not aware that pecl packages access policy has changed. thanks!
 [2014-12-30 16:52 UTC] garrettw (Garrett Whitehorn)
PEAR made the decision to move to Github independently. PECL has their own separate policies on this matter, and it looks like they use PHP's official Git server for most of their projects.