-
Notifications
You must be signed in to change notification settings - Fork 6
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
Added a few new things to the makeHAB script #8
Comments
Hi, thanks for the ideas!
Thoughts? |
Hi, you're welcome. I'll reply per item:
Shall i have a look how to implement these changes? |
Certainly. PRs are always welcome. |
I dídn't make a PR because i have to find out how that works. :-) However one addition to get it working properly: You'll have to remove the file org.ops4j.pax.url.mvn.cfg, otherwise the repository changes won't be processed. (See https://community.openhab.org/t/karaf-console-bundle-uninstall-and-bundle-update-not-running-as-expected/27444/22) |
also of note: openhab/openhab-distro#299 (comment) It might be time to join forces and work with that one. |
Maybe, but would that (new) script work with manual installations? I can only see folder names related to an apt-get install .... |
Hi,
i've added a few new things to my makeHAB script, which might be nice for the general script:
1: Check for root at the beginning of the script)
if [[ $(id -u) -ne 0 ]] ; then echo "Please run as root" ; exit 1 ; fi
2: Shutting down the openhab service after the files have been downloaded
3: setting permissions correctly so the service runs correctly after all the files have been placed.
4: Starting the openhab service
If you think these enhancements are too specific, please close this issue.
The text was updated successfully, but these errors were encountered: