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

Last merge makes 7 the minimum PHP version #1933

Closed
5 of 6 tasks
kaisermann opened this issue Jul 31, 2017 · 1 comment · Fixed by #1935
Closed
5 of 6 tasks

Last merge makes 7 the minimum PHP version #1933

kaisermann opened this issue Jul 31, 2017 · 1 comment · Fixed by #1935

Comments

@kaisermann
Copy link
Contributor

Submit a feature request or bug report


What is the current behavior?

Composer won't install if the dev/host machine doesn't have at least PHP 7. The illuminate/support is the "culprit".

Your requirements could not be resolved to an installable set of packages.
Problem 1
  - Installation request for doctrine/inflector v1.2.0 -> satisfiable by doctrine/inflector[v1.2.0].
  - doctrine/inflector v1.2.0 requires php ^7.0 -> your PHP version (5.6.30) does not satisfy that requirement.
Problem 2
  - doctrine/inflector v1.2.0 requires php ^7.0 -> your PHP version (5.6.30) does not satisfy that requirement.
  - illuminate/support v5.4.27 requires doctrine/inflector ~1.0 -> satisfiable by doctrine/inflector[v1.2.0].
  - Installation request for illuminate/support v5.4.27 -> satisfiable by illuminate/support[v5.4.27].

What is the expected or desired behavior?

Should it still work with PHP 5.6? Or should the readme be updated?


Bug report

Please provide steps to reproduce, including full log output:

Just have php 5.6 and try to composer install on the theme.

Please describe your local environment:

WordPress version: 4.8

OS: MacOS

NPM/Node version: X

PHP Version: 5.6

Where did the bug happen? Development or remote servers?

Remote server

Is there a related Discourse thread or were any utilized (please link them)?

Nope

@greatislander
Copy link

Would need a README update and https://github.com/roots/sage/blob/master/composer.json#L32 should be bumped too. If the intention is to drop 5.6 support.

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

Successfully merging a pull request may close this issue.

2 participants