Skip to content
This repository was archived by the owner on May 1, 2022. It is now read-only.

Conversation

@gameFace22
Copy link
Contributor

I have updated the file. Any errors,shout out. I would be obliged to correct it.
And also anything more needs to be added?
I thought of also adding 'Common errors while installing'
Ideas?

I have updated the file. Any errors,shout out. I would be obliged to correct it.
And also anything more needs to be added?
I thought of also adding 'Common errors while installing'
Ideas?
README.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not safe, a chown root:www-data and chmod 765 hackademic should be enough

@gameFace22
Copy link
Contributor Author

chown root www-data?
I am curious what is this for.
Explain me. Please.

So a new user should be created,right?

@northdpole
Copy link
Collaborator

I made a mistake here(apologies).
Here's how it works:
All the .php files which are the application logic which should be writable only by the user account who's installing hackademic.
The smarty-tmp and the challenges folder should be writable by the web server because that's where the user content and the smarty temporary files end up.
In order to install without having to supply the config file by yourself the folder needs to be writable by the web server but afterwards there is no use for this.
So, for the sake of user-friendliness we either have to alert the user to these permission changes. Or we could provide the configuration in a

 
section that the user can copy paste into the appropriate file.
What do you think?

@gameFace22
Copy link
Contributor Author

We can provide the config file in the pre section.
But,are we in need of config file after installing?

@northdpole
Copy link
Collaborator

yes we are. Tthe config.inc.php contains all the information for the db connection and the base path of the platform in the web server and the mode (Exhibition, debug, regular) among other things

Hackademic pushed a commit that referenced this pull request May 14, 2015
Extended information on deployment
@Hackademic Hackademic merged commit cae1a90 into Hackademic:next May 14, 2015
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.

4 participants