-
Notifications
You must be signed in to change notification settings - Fork 42
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
DSM 7: Clean up packaging files and make it conform with Synology guidelines #178
Comments
Updates: OpenHAB install in a non-persistent environment now. Basically removed all the start up scripts and read how the Karaf project suggests to start services. Works perfectly fine. Guess I will end up with a completely rewritten package. |
@thopiekar Thank you very much for your work. Let me know when you have. |
Got only one thing left: Using the existing Synology app to display the uuid and cloud id from the userdata directory. I'm going to create a request for support at Synology on this. I just can't find any documentation about app development. |
Hi @thopiekar, I am currently using openhab3 on my freenas server. I like it a lot and want now to install also this version on my fathers DS411 (armv5). I successfully installed your .spk for 3.0.0-SNAPSHOT but it will not start even the log says it has ("PID file created. done."). I just have java8 installed and since OH3 needs jre11 I think that is the reason. |
"Your" is not really correct. I'm contributor of openHAB but never put much effort to the Synology subproject. But you can do me a favour already.. can you access your dad's nas and run "uname -m"? Thanks! PS: Yes I'm planning to add OpenJDK for arm, too. Need "uname -m" for this. 😉 |
Sorry, I just came across this repo when I was looking for a way to install openhab on a synology and did not have a closer look. I thought you were the only contributor in here.
I just found a zulu build (zulu11.43.100-ca-jdk11.0.9.1-linux_aarch32sf.tar.gz) of java 11. Might this work? |
I'm going to use AdoptOpenJDK. Generally give me some more time please 😉 |
Sure, take your time. I've got now 3.0.0-RC2 to run with zulu java11. It took me some time (and netstat) to get to know that openhabs webservice only listens to 8443 and not 8080. But now I'm logging in and I'm fine. |
I guess skip working on the UI and symlink the UUID and cloud id files to the share as I do for the other files for now. @cniweb I have two wishes:
|
@MikeD23 Yeah, it is nice to see that OH3 has the same file structure. So no extra work for me here. Regarding Zulu builds: I need to twitch to these I guess. Was using AdoptOpenJDK but just noticed today that we (as OpenHAB family) recommend Zulu. |
@thopiekar can you complete your work and do a pull request, please? |
@cniweb: PR created, but there is still some work left. How are your thoughts on the points above? |
@thopiekar I remove the sponsor button. |
I agree with you that we will focus on stable code that is feature rich. |
@cniweb If I could get more involved into the project here. At least for the reviewing process. So yeah, that would be really nice 😉 Btw. I continued on the PR. I'm waiting there for feedback by some brave testers. |
@thopiekar |
Oh, yes. I'll rerun the build script it with the latest version of OpenHAB. |
I agree on the backward compatibility for OH2.
|
Just updated to DSM 7. |
We've been running our scripts with root rights for a long time.
With DSM 7 Synology prevents these packages to be installed and creates system users to define permissions safely.
I'm going to clean up the whole set of script and make it conform to Synology's guidelines. The download steps on preinstall will be kept, but I've got itchy fingers bundling all data in our spk.
I mean, logically you won't like to have our automatic next stable URL generator in any package like this.
I tried the used commands on my own and I already know what happens if someone likes to download 2.5.x, but once 3.x has been released, it will download 3.x. Our versioning here will be pointless.
However: This mechanism makes only sense if we detect major and minor and then download the latest version with the same major and minor version, but the most recent patch version. Per definition patch versions shouldn't break anything.
Finally, I'm wondering who tested this part of the script collection here.
But back to the topic: I've got DSM 7 beta installed here and will spend some time everyday on this. Will keep you posted.
Update OH Manager to display UUID and CloudConnector IDNOT GOING TO HAPPEN NOWPS: I'm also going to kill the "where to install" userdata functionality. Per Synology, you can register resources, like shares you need. Nevertheless, there is no way to say "hey, I need it optionally. I will tell you on install." Therefore, I'm going to create a "openhab" share fixed (as inspired by Docker). If someone likes to make it public, then give it "Everyone" permissions after install. And if you like to hide it, it can be done as you would do with the "docker" share. Something we can brainstorm still is a switch to link the files and directories into the share or make it "self-contained".
The text was updated successfully, but these errors were encountered: