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

command-line: fall back to LOUNGE_HOME env variable #402

Merged
merged 1 commit into from
Jun 13, 2016

Conversation

williamboman
Copy link
Member

@williamboman williamboman commented Jun 13, 2016

Makes it easier to use lounge with a different home configuration.

Closes #365.

@xPaw xPaw added Type: Feature Tickets that describe a desired feature or PRs that add them to the project. second review needed labels Jun 13, 2016
@maxpoulin64 maxpoulin64 merged commit ee5a644 into thelounge:master Jun 13, 2016
@xPaw
Copy link
Member

xPaw commented Jun 13, 2016

@williamboman Can you also add this to documentation on the site?

We used to support HOME but it was undocumented and with some undefined behaviour.

@xPaw xPaw added this to the 2.0.0 milestone Jun 13, 2016
@williamboman williamboman deleted the feat/use-LOUNGE_HOME-env branch June 13, 2016 18:35
@maxpoulin64
Copy link
Member

maxpoulin64 commented Jun 13, 2016

Little note for the next plans: it would probably be a good idea to have a variableto set the configuration file location, and be able to set the location for data/users/logs individually from within said config file. Some people (like me) like to have configs and data separate. It would also help make packages that respects the FSH properly without using symlinks (so /etc/thelounge.js for the config, /var/lib/thelounge/users/ for the users).

matburnham pushed a commit to matburnham/lounge that referenced this pull request Sep 6, 2017
…ME-env

command-line: fall back to LOUNGE_HOME env variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants