geerlingguy / ansible-role-php-pecl

Ansible Role - PHP PECL extension
https://galaxy.ansible.com/geerlingguy/php-pecl/
MIT License
32 stars 37 forks source link

Ubuntu 14 Pecl known bug #7

Closed upbeta01 closed 7 years ago

upbeta01 commented 8 years ago

http://askubuntu.com/questions/457242/download-of-pecl-xdebug-succeeded-but-it-is-not-a-valid-package-archive

geerlingguy commented 8 years ago

I haven't had an issue with this in building instances of Drupal VM or other projects that install extensions on 14.04... is this something that only affects one particular version of Ubuntu or can be worked around by running an apt-get update?

I'd rather not have the hundreds/thousands of people using this role suddenly waste an extra ton of bandwidth on every provision just to work around a particular Ubuntu bug.

upbeta01 commented 8 years ago

I could have had pasted the error in relation to the pull request comment, my bad. Basically it only affects one Ubuntu version but not sure if running "apt-get update" patches a fix for this bug -- but as far as my orchestration goes, it should have run apt-get update prior to installing your roles to the client host.

The objective for this pull request is to cater the ones who'll encounter the bug using the raw roles you have. Say like me, I have been using most of your roles for provisioning client hosts which is called via ansible-galaxy, as much as I would like -- I wanted to have the base form of it and pin it down to a specific version as when you'll be having some huge changes and update the repo, I won't have to re-install those roles again and re-edit the script just to place the " -Z " option in particular (if the new release still encounters the same error).

This is purely to preserve the scripts the way you have released it so the ones using it (if in case) encounters error won't be manually editing it.

geerlingguy commented 7 years ago

Linking http://www.antojose.com/solved-php-pear-install-pecl-uploadprogress-on-ubuntu-14-04-fix-workaround-error-could-not-extract-package-xml-download-succeeded-but-not-valid-package-archive — I just encountered this on an old project :P

geerlingguy commented 7 years ago

Merged, but made the install command a variable so it would stay as-is by default. Thanks!

upbeta01 commented 7 years ago

Nice. Glad to contribute! :)