Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add options for changing settings file location #786

Closed
wants to merge 2 commits into from
Closed

Add options for changing settings file location #786

wants to merge 2 commits into from

Conversation

Paul-Reed
Copy link
Member

@Paul-Reed Paul-Reed commented Feb 3, 2018

Enables the settings.php file to be moved outside of HTML route.
See this quick video.
This PR accomodates the following paths, but others could be very easily added for different operating systems, such as DietPi, etc;

  • home/pi/.emoncms/
  • home/pi/
  • or compatibility fallback of emoncms installation directory

Main advantages are;

  • Makes it easy to import/export as most things are in one directory
  • Moves settings file out of html path
  • Makes it easy to delete emoncms & reinstall afresh from git
  • Backwards compatible

Enables the settings.php file to be moved outside of HTML route.
Currently;
- home/pi/.emoncms/
- home/pi/
- or compatibility fallback of emoncms installation directory

Easy to add other locations for other operating systems.
Backwards compatible.
@Paul-Reed
Copy link
Member Author

Paul-Reed commented Feb 3, 2018

Submitted for discussion at this stage.
If this is the direction that we would want to pursue, I would suggest that I update the raspberry pi installation guide to reflect this, and possibly create a small bash script (in /scripts) to create the new directory structure, set ownership, and copy across default.settings.php to the new structure (as settings.php).
This would cut several steps out of the installation guide.

Paul

@pb66
Copy link
Collaborator

pb66 commented Feb 5, 2018

@Paul-Reed Paul-Reed closed this Feb 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants