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

Added 'binVersion' option to specify against which version to test - solves an issue on Ubuntu #41

Closed
wants to merge 4 commits into from

Conversation

xavierpriour
Copy link
Contributor

Hi,
On Ubuntu my php reports version '5.5.9-1ubuntu4.4'.
This fails the built-in PHP version check '>=5.4', because semver believes '1ubunbu4.4' is a pre-release tag, and pre-release tags do not match version range (unless they're explicitly listed, see npm/node-semver#99 (comment)).
So grunt-php aborts.

To solve it I added an option to specify the version to check against, and I set it to '*' (which matches), but it could also be used to ensure PHP>=5.5, if
Default is still >=5.4, so it is fully backward-compatible.

@sindresorhus
Copy link
Owner

Fixed in a dependency. Just reinstall this task.

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 this pull request may close these issues.

2 participants