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

Error on use jetty 9.3.0 #15

Closed
josivan opened this issue Jun 18, 2015 · 5 comments
Closed

Error on use jetty 9.3.0 #15

josivan opened this issue Jun 18, 2015 · 5 comments
Milestone

Comments

@josivan
Copy link

josivan commented Jun 18, 2015

I have updated to jetty-distribution-9.3.0.v20150612 and when I tried to run the jetty I got the message:

"Failed to resolve Jetty dependencies: [./lib/jsp/..jar]"

I have downgraded to the previous version of jetty (9.2.11).

Can you help me with that message?

Thanks

@update on 18/06. I analysed the code and I have noticed that line 38 of class net/sourceforge/eclipsejetty/jetty9/Jetty9LibStrategy.java might be the problem.

@mpetris
Copy link
Contributor

mpetris commented Oct 2, 2015

"The 9.2 release has switched to using the Apache version of Jasper for JSP and JSTL" so everything is in apache-jsp and apache-jstl and the directory "lib/jsp" isn't available anymore.

@quartex
Copy link

quartex commented Oct 8, 2015

Hi, I have the same problem with 9.3.3, I try to rename the folder (and duplicate it) but it does not work. Do you try to recompile the plugin or ..... to fix it ?
That's a shame for this problem because this is a great plugin !!
Thank you very much

@derkoe
Copy link
Contributor

derkoe commented Oct 8, 2015

It seems impossible to launch Jetty 9.x versions a similar manner. When I launch 9.2 with the libs in apache-jsp then JSPs won't work. This seems to be unfixable.

@mpetris
Copy link
Contributor

mpetris commented Oct 8, 2015

Sorry my earlier comment was misleading. It works if you stick with the libs/jsp for 9.2 and use lib/apache-jsp for 9.3. Obviously this involves separate JettyLibStrategies. I gave it a try and issued a pull request. The code works for my tests but maybe you want to have it in a different style. Anyway thanks for this great plugin! BTW I tested the WebSocket feature and it works very well.

@derkoe
Copy link
Contributor

derkoe commented Oct 10, 2015

See pull request #18

@derkoe derkoe closed this as completed Oct 10, 2015
@derkoe derkoe added this to the 3.8.1 milestone Oct 10, 2015
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

4 participants