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

Update branding.properties on distro upgrade #331

Closed
kaikreuzer opened this issue Nov 21, 2016 · 5 comments
Closed

Update branding.properties on distro upgrade #331

kaikreuzer opened this issue Nov 21, 2016 · 5 comments

Comments

@kaikreuzer
Copy link
Member

See https://community.openhab.org/t/new-folder-structure/16526/55

@xsnrg
Copy link

xsnrg commented Nov 22, 2016

@kaikreuzer it seems this should be part of the runtime, not the user data that should survive an upgrade, correct?

@kaikreuzer
Copy link
Member Author

Yes, this is where the etc folder was before the folder layout change. But if we move it back, we would need to reference it (and I don't think this is possible). So I rather tend to include it in a bundle.

@ThomDietrich
Copy link
Member

Without technical background: Shouldn't the branding.properties be a static file and the version number included from somewhere else? I am parsing the properties file to get the build number - feels wrong...

kaikreuzer added a commit to kaikreuzer/openhab-distro that referenced this issue Dec 8, 2016
…m files in userdata

Fixes openhab#331

Signed-off-by: Kai Kreuzer <kai@openhab.org>
@kaikreuzer
Copy link
Member Author

@ThomDietrich Well, this is how Karaf branding works. It is not possible to construct it dynamically at runtime.

I tried many different solutions for this problem and eventually came up with #345.
This allows us to write files at a first start after an upgrade to the userdata folder, so if necessary, we can upgrade Karaf-specific files in that folder (that users anyhow are not supposed to touch).

@ThomDietrich
Copy link
Member

Understood. In this case the old solution is acceptable. 👍

kaikreuzer added a commit that referenced this issue Dec 9, 2016
…m files in userdata (#345)

Fixes #331

Signed-off-by: Kai Kreuzer <kai@openhab.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants