velocity-tools icon indicating copy to clipboard operation
velocity-tools copied to clipboard

Draft PR for Jakarta ee10 jdk17

Open eduarddrenth opened this issue 1 year ago • 13 comments
trafficstars

For discussion on the diff, see https://issues.apache.org/jira/browse/VELTOOLS-202

eduarddrenth avatar Jul 18 '24 19:07 eduarddrenth

@arkanovicz I see at least two classes which have been copied from Tomcat. I wonder whether they are still required these days...

michael-o avatar Jul 19 '24 06:07 michael-o

@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 avatar Jul 19 '24 07:07 arkanovicz

@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?

michael-o avatar Jul 19 '24 08:07 michael-o

@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 avatar Jul 19 '24 08:07 arkanovicz

@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.

michael-o avatar Jul 19 '24 08:07 michael-o

@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.

michael-o avatar Jul 19 '24 13:07 michael-o

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.

eduarddrenth avatar Jul 20 '24 06:07 eduarddrenth

Why did you close? This is viable code.

michael-o avatar Jul 20 '24 06:07 michael-o

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

eduarddrenth avatar Jul 20 '24 11:07 eduarddrenth

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.

michael-o avatar Jul 20 '24 11:07 michael-o

Factored out a bit https://github.com/apache/velocity-tools/pull/21

michael-o avatar Nov 22 '24 14:11 michael-o

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?

eduarddrenth avatar Nov 22 '24 16:11 eduarddrenth

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

michael-o avatar Nov 22 '24 17:11 michael-o

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.

arkanovicz avatar Nov 03 '25 14:11 arkanovicz

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.

michael-o avatar Nov 03 '25 14:11 michael-o