-
Notifications
You must be signed in to change notification settings - Fork 12
Conversation
my bad, should target develop... |
README.md
Outdated
http://www.wampserver.com/ should be able to provide everything you need. | ||
|
||
|
||
## Deploy on shared web hosting services |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if it makes sense to keep this part. It has been written at a time where our website was hosted on an OVH PHP hosting where we didn't have any access to a shell.
So it was really hard to deploy Symfony on this platform and I put my hacks in these scripts to "ease" the deployment.
Finally, we moved to a VPS for performance and simplicity reasons.
If you don't use them, I am really encouraging you to:
- Remove this part of the documentation
- Remove the
./deploy
folder
In any case, I must warn you that even 4 years ago these scripts were not reliable to deploy.
And I think that today it can be only worse.
At least, if you are very nostalgic and want to keep these instructions+scripts, put an enormous disclaimer saying that this method is broken
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, thank you for your advice. We will remove that part.
Create database, user and grant access | ||
|
||
```mariadb | ||
CREATE DATABASE insalan; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you create database by this way, instead of using php app/console doctrine:database:create
as we did before ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The script was not able to connect to mariaDB since there was not a user 'insalan' in the database. Maybe because I was not logging in as 'insalan' on my test system.
I could try this week end or the next one (exams next week...)
Closing this, as all commits have been merged in develop. |
Rectify #74 and provide instruction for install on GNU/Linux targets.
However instructions for Windows have to be improved...