InsaLan / insalan.fr

InsaLan website.
https://www.insalan.fr
MIT License
9 stars 12 forks source link

PHP7 port #30

Open halfa opened 8 years ago

halfa commented 8 years ago

Should be tested against composer requirements.

halfa commented 8 years ago

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 commented 8 years ago

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

Lesterpig commented 8 years ago

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