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

Install instruc #79

Closed
wants to merge 3 commits into from
Closed

Install instruc #79

wants to merge 3 commits into from

Conversation

FoxyHawk
Copy link
Collaborator

@FoxyHawk FoxyHawk commented Feb 1, 2019

Rectify #74 and provide instruction for install on GNU/Linux targets.
However instructions for Windows have to be improved...

@FoxyHawk
Copy link
Collaborator Author

FoxyHawk commented Feb 1, 2019

my bad, should target develop...

@AntoineBvn AntoineBvn self-assigned this Feb 2, 2019
README.md Outdated
http://www.wampserver.com/ should be able to provide everything you need.


## Deploy on shared web hosting services
Copy link
Member

@superboum superboum Feb 2, 2019

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:

  1. Remove this part of the documentation
  2. 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

Copy link
Member

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.

@AntoineBvn AntoineBvn changed the base branch from master to develop April 9, 2019 15:44
Create database, user and grant access

```mariadb
CREATE DATABASE insalan;
Copy link
Member

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 ?

Copy link
Collaborator Author

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...)

@AntoineBvn
Copy link
Member

Closing this, as all commits have been merged in develop.

@AntoineBvn AntoineBvn closed this May 27, 2019
@AntoineBvn AntoineBvn deleted the install_instruc branch May 27, 2019 11:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants