php_pear cannot find available packages (tested with mongo package)

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Component/s: php
  • Labels:
    None
  • Environment:

    php cookbook v1.1.2

Description

Issue only seems to affect v1.1.2

php_pear "mongo" do
action :install
end

Cannot find mongo package in either PECL or PEAR repositories, even though package is available in manual install.

php recipe v1.1.0 works fine.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Cassiano Leal added a comment - - edited

It seems to be affecting all packages' installations via pear/pecl, so I bumped the priority to Major.

Here's the fix:
https://github.com/cassianoleal/php/tree/fix_pear_install

And this is the pull request for that:
https://github.com/opscode-cookbooks/php/pull/17

Show
Cassiano Leal added a comment - - edited It seems to be affecting all packages' installations via pear/pecl, so I bumped the priority to Major. Here's the fix: https://github.com/cassianoleal/php/tree/fix_pear_install And this is the pull request for that: https://github.com/opscode-cookbooks/php/pull/17
Hide
Permalink
Steven Danna added a comment -

Looks good. Marking for merge

Show
Steven Danna added a comment - Looks good. Marking for merge
Hide
Permalink
jtimberman added a comment -

Released in v1.1.4.

Show
jtimberman added a comment - Released in v1.1.4.
Hide
Permalink
Tom Usher added a comment -

This change appears to be causing an error for me on installing APC, reporting "Package apc not found in either PEAR or PECL." on Ubuntu 12.04 with this trace: https://gist.github.com/4510972
1.1.2 works without a problem.

Show
Tom Usher added a comment - This change appears to be causing an error for me on installing APC, reporting "Package apc not found in either PEAR or PECL." on Ubuntu 12.04 with this trace: https://gist.github.com/4510972 1.1.2 works without a problem.
Hide
Permalink
Cassiano Leal added a comment -

Have you tried "APC" instead? That's the actual package name in PECL:

$ pecl search apc
Retrieving data...0%
.Matched packages, channel pecl.php.net:
=======================================
Package Stable/(Latest) Local
APC 3.1.14 (beta) Alternative PHP Cache

Show
Cassiano Leal added a comment - Have you tried "APC" instead? That's the actual package name in PECL: $ pecl search apc Retrieving data...0% .Matched packages, channel pecl.php.net: ======================================= Package Stable/(Latest) Local APC 3.1.14 (beta) Alternative PHP Cache
Hide
Permalink
Tom Usher added a comment -

Good call, that seems to work, thanks!
Perhaps worth noting that it's now case sensitive, or removing the case sensitivity introduced by the 1.1.4 change.

Show
Tom Usher added a comment - Good call, that seems to work, thanks! Perhaps worth noting that it's now case sensitive, or removing the case sensitivity introduced by the 1.1.4 change.
Hide
Permalink
Tom Burdick added a comment -

This bug bit me today as I was reading the php docs here, http://docs.opscode.com/lwrp_php.html and found that "apc" didn't work after which chef gave a pretty baffling error message.

Trying sudo pecl install apc worked fine. Would be nice if either the doc examples are fixed and a strong note is placed in there or case sensitivity here is avoided. pecl install certainly didn't seem to care.

Show
Tom Burdick added a comment - This bug bit me today as I was reading the php docs here, http://docs.opscode.com/lwrp_php.html and found that "apc" didn't work after which chef gave a pretty baffling error message. Trying sudo pecl install apc worked fine. Would be nice if either the doc examples are fixed and a strong note is placed in there or case sensitivity here is avoided. pecl install certainly didn't seem to care.
Hide
Permalink
Cassiano Leal added a comment -

It would probably be best to open a new ticket since this one is closed.

Making the lwrp case insensitive is a very simple patch, but I'm not sure it's the way to go. The fact that there's no way for the user to specify whether he wants to pull the package from PECL or PEAR makes me wonder what would happen if there was a package with the same name but with different casing on each service. If you want to install the one from PEAR there would be now way to do that.

Note that I'm not a PHP dev and I'm not even sure that this is a valid reason for keeping the provider case-sensitive.

Show
Cassiano Leal added a comment - It would probably be best to open a new ticket since this one is closed. Making the lwrp case insensitive is a very simple patch, but I'm not sure it's the way to go. The fact that there's no way for the user to specify whether he wants to pull the package from PECL or PEAR makes me wonder what would happen if there was a package with the same name but with different casing on each service. If you want to install the one from PEAR there would be now way to do that. Note that I'm not a PHP dev and I'm not even sure that this is a valid reason for keeping the provider case-sensitive.

People

  • Assignee:
    Cassiano Leal
    Reporter:
    Guto Motta
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: