velocity-tools
velocity-tools copied to clipboard
Draft PR for Jakarta ee10 jdk17
For discussion on the diff, see https://issues.apache.org/jira/browse/VELTOOLS-202
@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...
@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...
I don't really maintain the jsp module. I released it unchanged last time, but if nobody can help maintain it, we can deprecate it.
@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...
I don't really maintain the jsp module. I released it unchanged last time, but if nobody can help maintain it, we can deprecate it.
That would mean that the JSP tags are dead, aren't they?
@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...
I don't really maintain the jsp module. I released it unchanged last time, but if nobody can help maintain it, we can deprecate it.
That would mean that the JSP tags are dead, aren't they?
I released it as is last time because I've been told that jsp hadn't change, so there was no real reason for the module not to be functional. But I never used jsp, never will, and I do not feel very concerned by its fate. Maybe ask the mailing list?
@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...
I don't really maintain the jsp module. I released it unchanged last time, but if nobody can help maintain it, we can deprecate it.
That would mean that the JSP tags are dead, aren't they?
I released it as is last time because I've been told that jsp hadn't change, so there was no real reason for the module not to be functional. But I never used jsp, never will, and I do not feel very concerned by its fate. Maybe ask the mailing list?
Yes, let's do so.
@eduarddrenth First of all, thank you for the good cooperation. I already see a few points I can cherry-pick and improve the current code on main, but first I need to take care of velocity master as a prep. Please stay tuned.
Thanks also, I enjoyed to play a small part in making velocity available to application servers for jakarta EE 10. I'll keep an eye open.
Why did you close? This is viable code.
Why did you close? This is viable code.
I thought you did not need it anymore. Is the process?
- prepare velocity master
- merge changes into my fork
- adapt or resubmit jakarta PR
- merge PR
- decide how to release jakarta version and release it
Why did you close? This is viable code.
I thought you did not need it anymore. Is the process?
* prepare velocity master * merge changes into my fork * adapt or resubmit jakarta PR * merge PR * decide how to release jakarta version and release it
You misunderstood. I need to keep it open for visibile and future changes.
Factored out a bit https://github.com/apache/velocity-tools/pull/21
At some point in the near future I need to decide how to include velocity deps in my jakarta apps. Either I wait for a jakarta release from the apache team, or I decide to keep my fork up-to-date and release to maven central myself. I prefer the first, but when will this release be?
At some point in the near future I need to decide how to include velocity deps in my jakarta apps. Either I wait for a jakarta release from the apache team, or I decide to keep my fork up-to-date and release to maven central myself. I prefer the first, but when will this release be?
I have expressed my opinion in this regard in the other PR already: https://github.com/apache/velocity-tools/pull/15#issuecomment-2403206831
Most of the standard containers are maintaining old versions for javax and switching to jakarta in their new versions.
I think that the easiest course of action is to do the same, because there is an underneath constraint about the target JVM version, and it will avoid us the hassle of maintaining different modules on the same sources. So my plan is:
- to release a 3.2 tools version on javax
- to release a 4.0 tools version on jakarta with a new velocity-master targeting the jvm 11
@michael-o , in this thread, you said:
There is another course of action, now that I think of it: maintain the tools 3.1.x on javax and switch on jakarta for 3.2+.
I am against that. Cutting off people for the years to come for a long waited update is not right. Both Jetty and Tomcat will support javax for quite some time.
But there is no reason why we cannot backport every new feature to the 3.x branch, so users wouldn't be cut off, would they?
And otherwise, I don't really know how to handle the JVM target difference.
Most of the standard containers are maintaining old versions for javax and switching to jakarta in their new versions.
I think that the easiest course of action is to do the same, because there is an underneath constraint about the target JVM version, and it will avoid us the hassle of maintaining different modules on the same sources. So my plan is:
- to release a 3.2 tools version on javax
- to release a 4.0 tools version on jakarta with a new velocity-master targeting the jvm 11
@michael-o , in this thread, you said:
There is another course of action, now that I think of it: maintain the tools 3.1.x on javax and switch on jakarta for 3.2+.
I am against that. Cutting off people for the years to come for a long waited update is not right. Both Jetty and Tomcat will support javax for quite some time.
But there is no reason why we cannot backport every new feature to the 3.x branch, so users wouldn't be cut off, would they?
And otherwise, I don't really know how to handle the JVM target difference.
The proposal and discussion was to drop javax altogether which I opposed. My proposal was to support both through on the fly code rewrite if possible. Your approach with backports works as well. At the end I wanted a low effort method to serve both.