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

[xmppclient] Set dependencies in the pom.xml to provided #11418

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

gkfabs
Copy link
Contributor

@gkfabs gkfabs commented Oct 19, 2021

As asked by @fwolter in #11247

I set all the dependencies in the pom.xml to provided for all the one which exists in features.xml, because it's okay to add a transitive dependency to features.xml, but not to the pom.xml

…one which exists in features.xml

Signed-off-by: Fabien Carrion <fabien@carrion.mx>
@fwolter
Copy link
Member

fwolter commented Oct 20, 2021

Looks good! Did you test the JAR on your system, just to be sure not only the compilation works, but also the dependency resolution on the system?

@kaikreuzer kaikreuzer added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Oct 20, 2021
@gkfabs
Copy link
Contributor Author

gkfabs commented Oct 22, 2021

I recompile openhab-xmppclient addons, regenerate an addon kar using openhab distro. And it works like a charm.

Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@fwolter fwolter merged commit 5eb004e into openhab:main Oct 22, 2021
@fwolter fwolter added the enhancement An enhancement or new feature for an existing add-on label Oct 22, 2021
@fwolter fwolter added this to the 3.2 milestone Oct 22, 2021
dschoepel pushed a commit to dschoepel/openhab-addons that referenced this pull request Nov 9, 2021
…one which exists in features.xml (openhab#11418)

Signed-off-by: Fabien Carrion <fabien@carrion.mx>
Signed-off-by: Dave J Schoepel <dave@theschoepels.com>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
…one which exists in features.xml (openhab#11418)

Signed-off-by: Fabien Carrion <fabien@carrion.mx>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…one which exists in features.xml (openhab#11418)

Signed-off-by: Fabien Carrion <fabien@carrion.mx>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…one which exists in features.xml (openhab#11418)

Signed-off-by: Fabien Carrion <fabien@carrion.mx>
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 for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants