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

Fix documentation for Maven jetty plugin for Jetty 12 #10809

Closed
hantsy opened this issue Oct 30, 2023 · 7 comments
Closed

Fix documentation for Maven jetty plugin for Jetty 12 #10809

hantsy opened this issue Oct 30, 2023 · 7 comments
Assignees

Comments

@hantsy
Copy link
Contributor

hantsy commented Oct 30, 2023

The official doc related to maven jetty plugin: https://eclipse.dev/jetty/documentation/jetty-12/programming-guide/index.html#get-up-and-running

I am using jetty in my project, when updating the jetty version to 12.0.2 from v11, the maven-jetty-plugin is missing in the Maven public repo. Check https://mvnrepository.com/artifact/org.eclipse.jetty/jetty-maven-plugin

@olamy
Copy link
Member

olamy commented Oct 30, 2023

you need to use the corresponding eeX version.
ee10 use this one https://repo.maven.apache.org/maven2/org/eclipse/jetty/ee10/jetty-ee10-maven-plugin/
you will find versions for ee8, ee9 and ee10

@hantsy
Copy link
Contributor Author

hantsy commented Oct 30, 2023

So the maven jetty plugin doc should be updated.

@olamy olamy changed the title Maven jetty plugin for Jetty 12 is not available Fix documentation for Maven jetty plugin for Jetty 12 Oct 30, 2023
@olamy olamy self-assigned this Oct 30, 2023
@olamy
Copy link
Member

olamy commented Oct 30, 2023

So the maven jetty plugin doc should be updated.

Good catch! we definitely need to fix this.
Would you be happy to provide a PR?

@hantsy
Copy link
Contributor Author

hantsy commented Oct 30, 2023

@olamy I will try to send a PR soon.

@joakime
Copy link
Contributor

joakime commented Oct 30, 2023

The moved jetty-maven-plugin coordinates are in the migration tables -> https://eclipse.dev/jetty/documentation/jetty-12/programming-guide/index.html#pg-migration-11-to-12

@JWT007
Copy link

JWT007 commented Jan 26, 2024

See #11329 - currently some of those coordinates are not correct.

@sbordet
Copy link
Contributor

sbordet commented Mar 9, 2024

Fixed by #11341.

@sbordet sbordet closed this as completed Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants