| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.1.14 |
| 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) | ||
[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-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 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.