Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Master (PHP 5.x) branch giving PHP 7 #5

Closed
BrendanTWhite opened this issue Jan 3, 2016 · 2 comments
Closed

Master (PHP 5.x) branch giving PHP 7 #5

BrendanTWhite opened this issue Jan 3, 2016 · 2 comments

Comments

@BrendanTWhite
Copy link
Contributor

Hi @yajra

When I do a fresh clean clone of the master branch and do vagrant up it does the base box import and a whole bunch of provisioning, and then it says php5-fpm: unrecognized service. And when I vagrant SSH in to the box and do service --status-all it has php7.0-fpm in the list but not php5-fpm.

So I'm fairly sure it's installing PHP 7 and not PHP 5.x. So I tried changing "php": ">=5.4" to "php": "~5.4" in composer.json, but that hasn't helped.

Also I'm guessing I probably need to specify version 0.3.x of the base box, not 0.4.0, because 0.4.0 is on PHP7. So I've added a config.vm.box_version = "~> 0.3" line right under config.vm.box = settings["box"] ||= "laravel/homestead" in scripts/homestead.rb, but that hasn't helped either. Even after I did a vagrant destroy && vagrant up, just to be sure. (You can see my attempts in this branch.)

Do you happen to have any suggestions on what I might need to do to get PHP 5 working again?

With many thanks,
@BrendanTWhite.

@BrendanTWhite
Copy link
Contributor Author

Actually, the config.vm.box_version in scripts/homestead.rb was the fix after all - I just needed to get my syntax right. It's "~0.3" not "~>0.3".

I'll clean up my branch and do a pull request.

@BrendanTWhite
Copy link
Contributor Author

...except that "~> 0.3" actually was the right syntax after all ... not sure why it wasn't working for me before.

Anyway, it's working now. I've put in a pull request for it.

Cheers,
Brendan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant