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

org.eclipse.equinox.jsp.jasper should not mandate EL RI requirement #725

Open
amergey opened this issue Dec 16, 2024 · 2 comments
Open

org.eclipse.equinox.jsp.jasper should not mandate EL RI requirement #725

amergey opened this issue Dec 16, 2024 · 2 comments

Comments

@amergey
Copy link
Contributor

amergey commented Dec 16, 2024

org.eclipse.equinox.jsp.jasper bundle requires EL RI
Import-Package: com.sun.el;version="3.0.0"

This requirement should be removed or at least set as optionnal as org.apache.jasper has it owns EL implementation (org.apache.el), not com.sun.el anymore

@merks
Copy link
Contributor

merks commented Dec 16, 2024

If I recall correctly this was needed to actually make the JSP help pages work with the versions of all that stuff actually being used. Help getting it working with newer dependency would be much appreciated; it's a challenge though...

@amergey
Copy link
Contributor Author

amergey commented Dec 16, 2024

But since the bundle requires jasper in version 9
org.apache.jasper.servlet;version="[9,10)"
jasper is not using el ri anymore (com.sun.el) but org.apache.el

I have tested an app with following dependencies to satisfy org.apache.jasper.servlet import


<dependency>
    <groupId>org.mortbay.jasper</groupId>
    <artifactId>apache-jsp</artifactId>
    <version>9.0.96</version>
</dependency>
<dependency>
    <groupId>org.mortbay.jasper</groupId>
    <artifactId>apache-el</artifactId>
    <version>9.0.96</version>
</dependency>

el ri is added as well but just to satisfy the import package, it is not used.

This seems to work as expected and looks like el ri should not be mandatory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants