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

Reporting not new enough PHP Version #165

Closed
texxasrulez opened this issue Oct 2, 2016 · 7 comments
Closed

Reporting not new enough PHP Version #165

texxasrulez opened this issue Oct 2, 2016 · 7 comments

Comments

@texxasrulez
Copy link

Within settings I get the following "Your version of PHP is too old! Please update to at least 5.3.0! You got the following version installed: 5.6.24-0+deb8u1". As can be seen, I have 5.6 installed.

@e-dschungel
Copy link
Contributor

The requirement was increased to PHP 7 in eb51b63 , but the error message was not changed, see #164 (comment)

@blind-coder: Is there a real reason for the increase to PHP7? "Why not" is not a good one as 7 is still not as widespread as versions >5.3: https://seld.be/notes/php-versions-stats-2016-1-edition and https://entwickler.de/online/php/php-7-verbreitung-248908.html
In the code I don't see PHP7 only features. Is #155 resolved? If you really like to go with PHP7 than it should be documented better.

@blind-coder
Copy link
Collaborator

I only have one machine and it is running PHP7. So I can't say for sure that everything works with earlier versions.

@blind-coder
Copy link
Collaborator

I have added an option to disable the version check. That should cover this issue.

@ghost
Copy link

ghost commented Oct 3, 2016

I have the same issue. It would be nice if you could fix this in the upstream version, as it makes rcmcarddav incompatible with the latest stable Debian 8.6.

@blind-coder
Copy link
Collaborator

As I said, Ihave no way to confirm PHP5 works. If you can check the issue_165 branch and get back to me, I can merge it into master and draft version 2.0.1.

@texxasrulez
Copy link
Author

Works beautifully. Thanks for your work.

@blind-coder
Copy link
Collaborator

Thanks, I have merged this into master.

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

3 participants