Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

PHP7 port #30

Open
halfa opened this issue Dec 5, 2015 · 3 comments
Open

PHP7 port #30

halfa opened this issue Dec 5, 2015 · 3 comments

Comments

@halfa
Copy link
Member

halfa commented Dec 5, 2015

Should be tested against composer requirements.

@halfa
Copy link
Member Author

halfa commented Apr 3, 2016

Tested today under debian/jessie:

sudo apt-get update && apt-get install php7.0 php7.0-curl php7.0-xml

This will replace the current php5 engine by php7.
We got full support for Symfony 2.8 hence we stick to it as 3.0 is not an LTS.

Here are the changes to the composer requirement:

index 9b80c4f..4900955 100644
--- a/composer.json
+++ b/composer.json
@@ -7,10 +7,10 @@
     "require": {
-        "php": ">=5.3.3",
+        "php": ">=7.0",
         "symfony/symfony": "^2.7",
         "doctrine/orm": "2.4.6",
-        "doctrine/annotations": "1.2.1",
+        "doctrine/annotations": "1.2.6",

The doctrine/annotations update is mandatory, see this thread

halfa added a commit that referenced this issue Apr 3, 2016
@halfa
Copy link
Member Author

halfa commented Apr 3, 2016

Apparently no issue, but I cannot say if perfomance-wise this is interesting to push in prod.

@Lesterpig
Copy link
Member

Should be safe, as there won't be any critical production event until November :-)

@halfa halfa removed their assignment Sep 29, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants