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
Remove openhab-transport-http feature dependencies #16202
Conversation
The openhab-transport-http feature dependency is redundant since OH 2.3 when it was added to the openhab-runtime-base feature dependencies. See: * openhab/openhab-core#344 * openhab#16181 (comment) Signed-off-by: Wouter Born <github@maindrain.net>
I just noticed openhab-transport-mdns was also added to openhab-runtime-base in OH 2 (openhab/openhab-core#81) so those feature dependencies could also be removed. WDYT? It probably got added because the MDNSAnnouncer is also used so OH can be discovered. See: /features/karaf/openhab-core/src/main/feature/feature.xml There may already be many add-ons that depend on these features without having these feature deps in their feature.xml file. |
Yes, the mdns feature should also be removed. |
The following features are already dependencies of openhab-runtime-base: * openhab-core-base * openhab-core-model-item * openhab-core-model-script * openhab-transport-mdns See also: openhab#16202 (comment) Signed-off-by: Wouter Born <github@maindrain.net>
The openhab-transport-http feature dependency is redundant since OH 2.3 when it was added to the openhab-runtime-base feature dependencies. See: * openhab/openhab-core#344 * openhab#16181 (comment) Signed-off-by: Wouter Born <github@maindrain.net>
* Remove redundant feature dependencies The following features are already dependencies of openhab-runtime-base: * openhab-core-base * openhab-core-model-item * openhab-core-model-script * openhab-transport-mdns See also: #16202 (comment) * Remove redundant openhab.tp-jackson and openhab.tp-jaxb feature dependencies Signed-off-by: Wouter Born <github@maindrain.net>
The openhab-transport-http feature dependency is redundant since OH 2.3 when it was added to the openhab-runtime-base feature dependencies. See: * openhab/openhab-core#344 * openhab#16181 (comment) Signed-off-by: Wouter Born <github@maindrain.net> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
* Remove redundant feature dependencies The following features are already dependencies of openhab-runtime-base: * openhab-core-base * openhab-core-model-item * openhab-core-model-script * openhab-transport-mdns See also: openhab#16202 (comment) * Remove redundant openhab.tp-jackson and openhab.tp-jaxb feature dependencies Signed-off-by: Wouter Born <github@maindrain.net> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
The openhab-transport-http feature dependency is redundant since OH 2.3 when it was added to the openhab-runtime-base feature dependencies.
See: