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

Consider OPENHAB_* on startup. Closes #408 #412

Merged
merged 1 commit into from
Jan 31, 2017

Conversation

heziegl
Copy link
Contributor

@heziegl heziegl commented Jan 30, 2017

if OPENHAB_CONF, OPENHAB_RUNTIME, OPENHAB_USERDATA or OPENHAB_LOGDIR are set upon start their values are now used instead of defaults.

Closes #408

if  OPENHAB_CONF, OPENHAB_RUNTIME, OPENHAB_USERDATA or OPENHAB_LOGDIR are set upon start their values are now used instead of defaults.
@mention-bot
Copy link

@heziegl, thanks for your PR! By analyzing the history of the files in this pull request, we identified @kaikreuzer, @theoweiss and @dvanherbergen to be potential reviewers.

@kaikreuzer
Copy link
Member

Thanks. Did you test whether it all still works correctly if you point the different variables to completely different folders?

@BClark09 How do the apt start scripts actually define the folders? Aren't they using the oh2_dir_layout script?

@BClark09
Copy link
Member

BClark09 commented Jan 31, 2017

@kaikreuzer
Copy link
Member

Ok, thanks - so this shows that it is pretty safe to define custom folders :-)

@kaikreuzer kaikreuzer merged commit 1d093e8 into openhab:master Jan 31, 2017
@kaikreuzer kaikreuzer modified the milestone: 2.1.0 Jun 25, 2017
@kaikreuzer kaikreuzer added the enhancement An enhancement or new feature label Jun 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants