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.

Sign up for GitHub

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

Jump to bottom

jetty-slf4j-impl missing from BOM #7327

Closed
mleegwt opened this issue Dec 22, 2021 · 1 comment · Fixed by #7328
Closed

jetty-slf4j-impl missing from BOM #7327

mleegwt opened this issue Dec 22, 2021 · 1 comment · Fixed by #7328
Labels
Bug For general bugs on Jetty side

Comments

@mleegwt
Copy link

mleegwt commented Dec 22, 2021

Jetty version(s)
Jetty 10 and 11

Java version/vendor (use: java -version)
Any

OS type/version
Any

Description
The package jetty-slf4j-impl is missing from the BOM.

How to reproduce?
Check the pom.xml in the jetty-bom project.
@mleegwt mleegwt added the Bug For general bugs on Jetty side label Dec 22, 2021
joakime added a commit that referenced this issue Dec 22, 2021
@joakime
Issue #7327 - Add missing entries to jetty-bom
ed39b19 
Signed-off-by: Joakim Erdfelt <[email protected]>
@joakime joakime linked a pull request Dec 22, 2021 that will close this issue
Issue #7327 - Add missing entries to jetty-bom #7328
Merged
@joakime
Copy link
Contributor

joakime commented Dec 22, 2021

Opened PR #7328 - reviewed all maven modules in our build and added the missing entries.

joakime added a commit that referenced this issue Dec 22, 2021
@joakime
Issue #7327 - Add missing entries to jetty-bom (#7328)
532e63b 
Signed-off-by: Joakim Erdfelt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For general bugs on Jetty side
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #7327 - Add missing entries to jetty-bom jetty/jetty.project
2 participants
@joakime @mleegwt