-
Notifications
You must be signed in to change notification settings - Fork 629
PHP version #16
Comments
Sean, Can you paste your default.pp file? On Sat, May 18, 2013 at 5:40 PM, Sean Downey notifications@github.comwrote:
|
Sure I manually added the php modules as they were blank - I put a comment in on another ticket about that. Thanks exec { 'apt-get update': class { 'apt': package { ['python-software-properties']: file { '/home/vagrant/.bash_aliases': package { ['build-essential', 'vim', 'curl']: class { 'apache': } apache::dotconf { 'custom': apache::module { 'rewrite': } apache::vhost { 'www.example.org': apt::ppa { 'ppa:ondrej/php5': } class { 'php': php::module { 'php5-cli': } class { 'php::devel': class { 'php::pear': class { 'xdebug': } xdebug::config { 'cgi': } class { 'php::composer': } php::ini { 'default': class { 'mysql': |
I've tested this several times on local and on prod, both work correctly. Maybe you got one of my earlier buggy releases. Care to try again? |
@jtreminio I got the same again 5.3.10. I used Ubuntu Precise 64 Bit (12.04.2 LTS) in case that was the issue? |
@jtreminio Seems that in this case the provisioning installs PHP first before the ppa is registered. This should do the trick:
|
Actually I see what happened - I had this before:
but removed it in a previous commit. It was the only way I found I could get the PPA to install, run apt-get update, then install php5.4 @frastel I'll see if your code does the same! |
Thanks @frastel, per documentation that's what I should have been using. I had tried to use it before, but I had been doing backwards! @seandowney it should work now. Try again? |
Fixed in 46bfada |
Cool, I added that in manually and trying again. |
That did the trick - nice one. Thank you both |
The "Use PHP 5.4" option doesn't seem to work. PHP 5.3.10 has install for me when I run "php -v"
The text was updated successfully, but these errors were encountered: