Support for PECL zend extensions, and better file name handling.

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Unknown Unknown
  • Resolution: Fixed
  • Component/s: php
  • Labels:

Description

The pecl (pear) provider assumes all extensions are of the form package_name.so, which is not always the case.
Also it has no support for extensions which must be loaded by the zend extension loader.

I've fixed this in my branch: https://github.com/leth/cookbooks/tree/pecl-zend

I'm pretty new with ruby, so any coding style/idiom suggestions would be welcome.

Thanks,

P.S. Pull request incoming.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Marcus Cobden added a comment -

Here you go: https://github.com/opscode/cookbooks/pull/359

Show
Marcus Cobden added a comment - Here you go: https://github.com/opscode/cookbooks/pull/359
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

1) The resource should have the zend_extensions attribute added to it as well with a reasonable default value.

2) What is Set['xdebug.so']? Is this a working example?

# install the xdebug pecl
    php_pear "xdebug" do
      zend_extensions Set['xdebug.so']
      action :install
    end

3) You should indent your blocks so it is more obvious they're blocks, see below this line:

extensions = Hash[ files.map { |filepath|
Show
Bryan McLellan [Opscode] added a comment - 1) The resource should have the zend_extensions attribute added to it as well with a reasonable default value. 2) What is Set ['xdebug.so'] ? Is this a working example? # install the xdebug pecl php_pear "xdebug" do zend_extensions Set['xdebug.so'] action :install end 3) You should indent your blocks so it is more obvious they're blocks, see below this line: extensions = Hash[ files.map { |filepath|
Hide
Permalink
Marcus Cobden added a comment -

1) Whoops, I suspect I forgot to add the changes to that file.

2) I'll comment this, and change it to expect an array, rather than a set, since it's not going to be a real performance problem.

3) Thanks. I don't know why I missed that.

I'll get to work on this on monday!

Show
Marcus Cobden added a comment - 1) Whoops, I suspect I forgot to add the changes to that file. 2) I'll comment this, and change it to expect an array, rather than a set, since it's not going to be a real performance problem. 3) Thanks. I don't know why I missed that. I'll get to work on this on monday!
Hide
Permalink
Marcus Cobden added a comment -

I've updated the pull request.

Show
Marcus Cobden added a comment - I've updated the pull request.
Hide
Permalink
Marcus Cobden added a comment -

I've made the changes suggested.

Show
Marcus Cobden added a comment - I've made the changes suggested.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Looks good, marked to be merged.

Show
Bryan McLellan [Opscode] added a comment - Looks good, marked to be merged.
Hide
Permalink
Joshua Timberman [Opscode] added a comment -

This is merged in the new php repository, https://github.com/opscode-cookbooks/php and will be included in the next release. Thank you!

Show
Joshua Timberman [Opscode] added a comment - This is merged in the new php repository, https://github.com/opscode-cookbooks/php and will be included in the next release. Thank you!
Hide
Permalink
Marcus Cobden added a comment -

Sorry, I failed to include the changes to the template file.

Show
Marcus Cobden added a comment - Sorry, I failed to include the changes to the template file.
Hide
Permalink
Marcus Cobden added a comment -

I've opened another pull request against the new repository.
Sorry about that!

https://github.com/opscode-cookbooks/php/pull/3

Show
Marcus Cobden added a comment - I've opened another pull request against the new repository. Sorry about that! https://github.com/opscode-cookbooks/php/pull/3
Hide
Permalink
Joshua Timberman [Opscode] added a comment -

Ah, thanks for catching that Marcus. I merged that pull request. Cheers!

Show
Joshua Timberman [Opscode] added a comment - Ah, thanks for catching that Marcus. I merged that pull request. Cheers!
Hide
Permalink
Joshua Timberman [Opscode] added a comment -

Released in v1.1.0

Show
Joshua Timberman [Opscode] added a comment - Released in v1.1.0

People

  • Assignee:
    Unassigned
    Reporter:
    Marcus Cobden
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: