You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@freemarker.apache.org by "ppodgorsek (via GitHub)" <gi...@apache.org> on 2023/07/07 21:43:25 UTC
[GitHub] [freemarker] ppodgorsek opened a new pull request, #94: FREEMARKER-218 - Add jakarta.servlet support
ppodgorsek opened a new pull request, #94:
URL: https://github.com/apache/freemarker/pull/94
**What has been changed in this PR:**
* Migrated from Javax to Jakarta
* Java upgrade: the minimum Java version has been increased to 17.
* Jakarta requires version 11+
* Spring 6 requires version 17+
* As most vendors will stop supporting Java 11 next year, it makes sense to switch to Java 17.
* [The JVM's Locale providers have been updated in versions > 9](https://openjdk.org/jeps/252), which required minor changes in the localised tests
* Hamcrest now provides the matchers that were created in the FreeMarker tests, the latter have therefore been removed to avoid duplication and reduce maintenance
* RMIC: the RMIC task doesn't support Java > 15 unfortunately. The associated tasks have been removed.
* The Ivy configuration has been cleaned up, all dependency versions are now declared in a consistent manner via XML entities
* The README has been updated accordingly
**Given the scope of the changes, I would suggest we create a `2.4-gae` branch and change the target of this PR before merging it. All files and documentation in this PR already refer to these changes as being part of release 2.4.**
I tried performing this change on branch `3` but it is incredibly far from `2.3-gae`'s head. Furthermore, the git history of branch `3` seems to be completely messed up, some gradle files are linked to old Java files, other files have been renamed without it being tracked. To avoid regressions, I would advise to rename `3` to `3-deprecated`, to create a new `3` branch from `2.4`, and to cherry-pick new changes as needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1627195405
Actually... the other pending PR-s will enter merge conflict status if I merge the Gradle PR... So, forget that, as that will happen when I managed to reduce the set of pending PR-s to just that.
Meanwhile, my question is if you are willing rework this to be a non-breaking change. As you might seen in the Ant file, we compile different set of classes with different dependencies (including different java version). And then, on runtime we use Java reflection to decide which version of a support class to use. Since these things (Servlet API, Jython API, etc.) are provided dependencies as far as we are concerned, we just have to "magically" work in the random environment we are dropped into.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1906428501
Silly question, how do I build it so that it puts it in my local gradle cache or maven local? So I can correctly reference it from my gradle build file with `'org.freemarker:freemarker-gae:2.3.33-SNAPSHOT'`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1908169437
I meant... this repo, not staging:
```
<repository>
<id>apache-snapshot-repository</id>
<url>https://repository.apache.org/content/repositories/snapshots/</url>
<releases><enabled>false</enabled></releases>
<snapshots><enabled>true</enabled></snapshots>
</repository>
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783774704
So, we have to decide which of the end results we want. Those are, multiple artifact with different classifier (or version), or separate freemarker.servlet and freemarker.jakartaservlet packages? Or, third, just break stuff, and require users to migrate to Jakarta. Then we can decide how to achieve that end result. (And I will be surprised if the current ancient Ant build can't do it. I mean, yes, we do migrate away from Ant, but let's not act as if that's a blocker, until we know it.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2004441644
Just to be clear (for anyone reading), what you are testing is actually the other Jakarta PR, https://github.com/apache/freemarker/pull/104. That's what's in the 2.3.33-SNAPSHOT build that I have linked earlier, not this PR where we are now commenting.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ppodgorsek commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ppodgorsek (via GitHub)" <gi...@apache.org>.
ppodgorsek commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1626962891
Thanks for the feedback, @ddekany
I'm not sure I understand how you'd like these changes to be incrementally merged.
The Jakarta artifacts have been compiled with Java 11 so even separating both wouldn't greatly simplify this PR.
The Java upgrade would anyway still require:
* JVM Locale changes
* RMIC support removal
* Ivy configuration updates
* Changes in the README
I would agree with you about BC once the project uses a "modern" modularised build system such as Gradle, which would allow us to publish two sets of JARs, one with javax and one with jakarta.
In the meantime, that's not the case for the `2.3-gae` branch that everyone relies on, and publishing one artifact with both would actually be more of a hassle for the dependency management of projects relying on FreeMarker.
You have a point about the versioning, I agree these changes make more sense as part of release 3.
I've been a massive supporter of FreeMarker for almost two decades but, having spent the last week trying to perform what I thought would be a very straightforward upgrade, I must admit that keeping BC at all costs is harming this project.
From an outside perspective, a lot of the code relies on magic / insider knowledge. 😄 I had to spend a lot of time understanding what the legacy configurations were used for in order to understand what were breaking changes from upgrades such as the JVM rather than simple adjustments. This very steep learning curve makes it very hard for anyone else to contribute, in turn increasing risk if support and maintenance end up being on one person's shoulders only. (yours)
The only reason I powered through is because, without Jakarta, FreeMarker is likely to die alone (frameworks such as Spring drive the adoption/decline of many other projects) and that would be a pity. Due to the Jakarta migration, Spring 6 shipped with a very minimalistic version of FreeMarker, which greatly diminishes its value unfortunately.
Branch `3` is missing features (from the top of my head `withArgs` is one of them) so even if the git history has diverged, there is still a big risk of breaking backwards compatibility from a feature perspective.
FreeMarker being a templating engine, I believe its main focus should be on feature parity/compatibility/BC rather than API BC.
Many of the artifacts relying on it on [mvnrepository.com](http://mvnrepository.com/) seem to be packaging it as part of web or email frameworks, without necessarily extending FM capabilities. (admittedly, I haven't checked all of them) From my own experience too, I've never decided to use the FreeMarker JAR because it had that one class that would be so useful on my project. Instead, its value lies in its capabilities.
I'll keep this PR open (I'll adjust the version number) and will start packaging and distributing my fork in the meantime, as this will unlock a smoother upgrade to Spring 6 for many projects.
I'll continue keeping my fork up to date and will be opening PRs for you as new changes are done on my side. (a wider adoption of Java 5 generics and replacement of a lot of code by standard Apache Commons libraries will be next on my to do list) Java 9 modules will also be a good way to enforce what can be used and extended outside of FreeMarker, this should greatly help with BC of selected parts of the project.
I wanted to add a big thank you for your great work on this project. As mentioned, I've been using it for many years and always thought it was awesome, so please see the above as constructive feedback rather than criticism. I really value all the hard work that has gone into FreeMarker.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] dtrunk90 commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "dtrunk90 (via GitHub)" <gi...@apache.org>.
dtrunk90 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1629809293
https://github.com/apache/freemarker/pull/95
Not sure if this is the way you wanted it and even if it works. It's not tested yet.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1630197646
@dtrunk90 Yes, I guess something like #95 will work.
@ppodgorsek Sorry about this. It's better to discuss more involved changes on the dev list before investing into them.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1908163739
Note that this is included in 2.3.33-SNAPHSOT in the Apache Maven snapshot repo:
https://repository.apache.org/content/groups/snapshots/org/freemarker/freemarker/2.3.33-SNAPSHOT/
<repository>
<id>apache-staging-repository</id>
<url>https://repository.apache.org/content/repositories/staging/</url>
</repository>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "bmarwell (via GitHub)" <gi...@apache.org>.
bmarwell commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783751638
That's not true. With Maven we can create an additional artifact with a transformer which just replaces the imports. That's probably true for Gradle as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "dtrunk90 (via GitHub)" <gi...@apache.org>.
dtrunk90 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783769607
> From the perspective of the JVM, you end up having a `freemarker.ext.servlet` that's not changed, and a `freemarker.ext.jakartaservlet` package that's a basically a copy of the `freemarker.ext.servlet` with some changes. That's what I called a duplicate. What technique is best to achieve that result, that's open question. Certainly it should be done as part of the build process.
Well, some provide 2 artifacts (1 for javax, 1 for Jakarta) and others are going with the future and just force their users to migrate to Jakarta EE. But for the time being I'm just using the Tomcat Migration tool to patch incompatible dependencies. So, for this discussion I'm out. It's up to the project maintainers to decide which way they want to go. I totally understand the pain as this simple namespace change has one of the biggest impacts in the entire Java history.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "Karthi4187 (via GitHub)" <gi...@apache.org>.
Karthi4187 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2027419954
Hi, any updates on the actual release date of 2.3.33?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1905619186
Pushed the change from PR #104. At least based on the JUnit tests, `JspTaglibs` will works.
So, now the freemarker jar artifact contains `freemarker.ext.jakarta.servlet`, and `freemarker.ext.jakarta.jsp`, in additionally to the old javax Serlvet/JSP support classes. People who need to switch to Jakarta has to search-and-replace their code, and use the new packages.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "skuzey (via GitHub)" <gi...@apache.org>.
skuzey commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2003369390
I will prioritise its testing
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783769414
3 is irrelevant here. Treat is as vapor ware until it produced some candidate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ppodgorsek commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ppodgorsek (via GitHub)" <gi...@apache.org>.
ppodgorsek commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1630232175
That's ok. Looking at the code of versions 2.x/3.x and the discussion we had here were insightful.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "yunkaiOr2 (via GitHub)" <gi...@apache.org>.
yunkaiOr2 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2167507643
don`t work, 2.3.33 or 2.3.34-SNAPSHOT
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "skuzey (via GitHub)" <gi...@apache.org>.
skuzey commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1983396918
Hi, any updates on the actual release date for the jakarta support?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1858791651
The mail discussion was in favor of adding a new package in the monolithic artifact, if we count the answers. So probably that will be it. But we do this after the other PR-s were settled, ending with the Gradle one, as the solution to this will (certainly) involve the build tool.
BTW, I check what's with Spring integration. At a point they have switched to Jakarta, and has simply removed some of the Servlet/JSP related features from the spring-freemarker (and I don't think may will miss them). So, whatever we do, Spring integration is not affected.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "firatkucuk (via GitHub)" <gi...@apache.org>.
firatkucuk commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1778914985
@ppodgorsek I have created a temporary fork of freemarker based on your PR. We are gonna use that one internally until we have an actual Jakarta compatible release of freemarker. Thanks :pray:
https://github.com/bloomreach/freemarker
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2036888620
Same here
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "georgecao (via GitHub)" <gi...@apache.org>.
georgecao commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2081760748
when will version 2.3.33 be released?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783771861
Yeah, actually, there's the othe way of "duplicating", where we have freemarker:freemarker:2.3.33, and freemarker:freemarker:2.3.33-jakarta artifas. Kind of like Guava does it. Or use Maven classifiers. Had to research which have what gotchas in practice.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783749190
Unfortunately, we will have to duplicate everything servlet-related, into its own package. Otherwise we break apps that expect the original Servlet API. So, if someone wants to work on a PR, that's what it should do. We can't just change the existing classes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1904341898
Yes, we do use JSP Taglibs in this manner
```
<#assign c=JspTaglibs["/WEB-INF/c.tld"]>
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "firatkucuk (via GitHub)" <gi...@apache.org>.
firatkucuk commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1777066697
Our codebase is dependent on FreemarkerServlet this PR seems a good solution. What are the plans for the Jakarta upgrade?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1626899840
Also, there's an old PR that migrates 2.3.x from Ant/Ivy to Gradle. That should be merged first, and then some of these changes are not applicable anymore. I know, why it's the for so long there unmerged... well, I put that on the top of my FM pipeline, and, hoping it will be done in week or so.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1800287771
So, we have to chose out which end result (ignore the "how" for now):
1. We can copy the `freemarker.ext.servlet` package into `freemarker.ext.jakartaservlet`, and we will have it the normal artifact.
2. We can have an additional artifact variant (let's say via Maven classifier "jakarta"), that still uses `freemarker.ext.servlet` package, but that links to the Jakarta classes.
Possibility 1 has the pro that we don't have to publish one more artifact, and also, then users don't have to fiddle with dependency management to chose the "jakarta" classifier artifact. But it has the con, that any existing dependent Java code that used `freemarker.ext.servlet` so far, and wants to migrate to a Jakarta Servlet container, has to be modified to link to `freemarker.ext.jakartaservlet` instead. That sounds quite bad, however, the same dependent Java code likely has to be modified anyway, to link to Jakarta Servlet classes. `web.xml`-s that refer to `freemarker.ext.servlet.FreemarkerSerlvet` also have to be modified, if someone uses a Jakarta container.
Actually, I will bring it up on the mailing list... if you are also there, please answer there.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1905994452
It's in 2.3.33, which I guess (no promises) will be released late February. You can help with testing/using this new feature, and trying to break it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1627052476
FreeMarker always supported multiple version of Java, and multiple version of the same library (especially multiple version of the servlet API) in the same single jar. So you can add Jakarta support without removing anything, and breaking anything. Gradle doesn't change any of that (though with Gradle 2 is at least modularized on source code level). The only reason FreeMarker 2 is not modularized (unlike 3) is backward compatibility.
BC is a huge pain, yes. Most users just add it as a single simple dependency, write templates, and don't care why it's working behind the scenes. But, most BC pain actually comes from issues in the template language itself, which can't be fixed without heavily breaking backward compatibility with existing templates. That's what 3 is really about (would be, if I invest the time...), although along the way it also fixes lot of historical mistakes in the API. (And 3 lives in different packages, it's not a problem if some dependency of your project uses 2, and some other uses 3.)
I'm not sure what's up with Spring 6 support. Is Spring MVC FreeMarker support depend on Servlet API? Because JSP 2 "Model 2" is really old school at this point. (Besides, FreeMarker support was always a second class check box feature in Spring. First because the official choice was JSP, and then because of Thymeleaf. The only way to have proper Spring support would be to put the resources on that on the FreeMarker side.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1629711870
Again, the Gradle PR is irrelevant in this. It will not change anything in this regard in the 2.x branch, as the binary output has to be the same as it was with Ant: a single artifact. All the dependencies of that are optional. But we can't even declare them like that in the Maven POM, as it doesn't have the concept of having optional dependencies of multiple versions of the the same thing. So, just do it with Ant, it's just copy-paste-modify of what we already do.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ppodgorsek closed pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ppodgorsek (via GitHub)" <gi...@apache.org>.
ppodgorsek closed pull request #94: FREEMARKER-218 - Add jakarta.servlet support
URL: https://github.com/apache/freemarker/pull/94
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1626876932
That's a lots of **unrelated** changes in a single PR, which will make it very difficult (like slow) to merge. Admittedly, that's also because I have very limited time for FM recently, but that what we have. Given that some of these changes are backward compatible, it would be good the break this up, be done with this set of changes piece by piece.
The other thing is, the new Jakarta API can be supported next to the old, so it need not be BC breaking, and things can stay a 2.3.x then. Supporting multiple Java versions was done through the whole history of the project (different java files are compiled with different versions), so that's a not a problem either. Surely it adds some complexity, but also removes the BC obstacle (i.e., if we go for 2.4, there's the logical desire to pack in more changes than supporting Jakarta).
`3` and `2.3-gae` source codes are too different for Git merges to work. So features for 2.3 are forward ported manually. 2.3 is not that "far ahead" much. Recreating 3 from 2.4 would be orders of magnitude more work than manually applying your changes on `3`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "skuzey (via GitHub)" <gi...@apache.org>.
skuzey commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2036692853
We have integrated 2.3.33 into our codebase and facing no issues so far. didn't have comprehensive testing yet though.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1984097769
That would be the release date of 2.3.33. It's waiting for more testing and the voting at this point, no blocker missing features/fixes. So few more weeks, I think. But, please test it if you can. You don't even have to build it, as it's in the Apache snapshot repo (see above).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2085351815
> when will version 2.3.33 be released?
Well, last time it was late February... yeah. My current guess is that we can start voting this week, and then the voting passes in 1-2 weeks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783767477
From the perspective of the JVM, you end up having a `freemarker.ext.servlet` that's not changed, and a `freemarker.ext.jakartaservlet` package that's a basically a copy of the `freemarker.ext.servlet` with some changes. That's what I called a duplicate. What technique is best to achieve that result, that's open question. Certainly it should be done as part of the build process.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783795735
Well, during the December spring (latest) I will probably start trying https://github.com/apache/tomcat-jakartaee-migration, as that even works with the current Ant solution. Hoping it will be done in a few hours, that surely worth it. (There are many other things that users are waiting for, and a few weeks a year to grind on them, so... you know, priorities.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1906356301
That's just how it always worked at this project... 2.4.0 is reserved for minor backward compatibility breaks. 2.3.x-es are 2.3.0 compatible (except that minimum dependency requirements can increase, in a conservative way). To clarify, this version still support legacy Servlet/JSP as well, so it's backward compatible.
Yeah, maybe in the future we should switch to semver...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ppodgorsek commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ppodgorsek (via GitHub)" <gi...@apache.org>.
ppodgorsek commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1628298613
I fully agree about BC for FreeMarker features, there has never been any issue with that part.
I can do some of the adjustments.
About Spring support, I'm not sure I understand what you mean. It has always been very straightforward to configure and use FreeMarker view resolvers, including in combination with other tools such as Tiles.
About dependency management, I won't do that part until Gradle has been put in place and different dependencies can be managed properly via modules. Abusing optional dependencies is a bad practice and makes things harder for users, especially on large projects.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] dtrunk90 commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "dtrunk90 (via GitHub)" <gi...@apache.org>.
dtrunk90 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1628381808
BC and the pending gradle PR are the reasons I just opened the jira issue without providing a PR myself. Why spending hours of adding a feature if everything needs to be re-done when the gradle switch was finally done? Since there are tools like https://github.com/nebula-plugins/gradle-jakartaee-migration-plugin it's not a blocker anyway.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "bmarwell (via GitHub)" <gi...@apache.org>.
bmarwell commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783782483
> Maybe you could also just integrate the Migration tool
That's exactly what I suggested. No new in packages. Maven transformers do exactly the same thing.
One project, two artifacts, no new packages.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "dtrunk90 (via GitHub)" <gi...@apache.org>.
dtrunk90 commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783782143
> So, we have to decide which of the end results we want. Those are, multiple artifact with different classifier (or version), or separate freemarker.servlet and freemarker.jakartaservlet packages? Or, third, just break stuff, and require users to migrate to Jakarta. Then we can decide how to achieve that end result. (And I will be surprised if the current ancient Ant build can't do it. I mean, yes, we do migrate away from Ant, but let's not act as if that's a blocker, until we know it.)
Maybe you could also just integrate the Migration tool into the ant build process and let it do the work for you. It's just a java class to call which will create a 2nd patched artifact: https://github.com/apache/tomcat-jakartaee-migration
Not sure exactly how and if it's possible for a project itself. But it definitely works if you have freemarker as a dependency (or jar file) in your project for example.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1985354783
FYI we've been using/testing it for a few weeks now and we've had no issues
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ppodgorsek (via GitHub)" <gi...@apache.org>.
ppodgorsek commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1783768229
@firatkucuk As you can see in the above discussion, a few points were raised:
* upgrading to Jakarta means many other things must change, (that explains why the PR is so large unfortunately)
* for some weird reason, `javax`/`jakarta` **must** have special treatment vs any other dependency, instead of treating it like any breaking change introduced by a major version upgrade, (with the same logic, a case could easily be made to treat any other dependency in a similar way)
* the git history between versions 2 and 3 has diverged, (branch `3` did not contain all features of branch `2.3` when I worked on it, so it is likely that some fixes and new features were missed when double-merging to both release branches)
* code is very difficult to read and maintain.
It's unlikely FreeMarker 3 will be production-ready when/if it is released, as the unclean git history will very likely introduce regressions and bugs. Added to the fact that projects still relying on `javax` were preventing us from modernising our own, especially if `jakarta` support was very low in their priorities, we have therefore replaced FreeMarker by Thymeleaf in all projects in our organisation, bar one (but that's in progress), and have made private any public repositories based on FreeMarker, to encourage the community to switch to more modern libraries instead. It is sad to have to move away from a tool which had been so useful for decades, but a move that seems necessary unfortunately.
Probably not what you wanted to read, but just my two cents.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1905634887
Awesome, thank you.
Any ideas when this will be released? Is there anything I can do to help?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1906075195
Why not increase the version to 2.4.0?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2103223948
FYI, voting on 2.3.33 release has started yesterday:
https://lists.apache.org/thread/r44nkyddvdqb2bor38rok4cypk9jms64
Let's hope it passes...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2167603161
You must use freemarker.ext**.jakarta**.servlet instead of freemarker.ext.servlet. It's not enough to just update your FreeMarker dependency. We support both pre-Jakarata and Jakarta environments in the same artifact, so the name of the non-Jakarta classes did not change.
Also note that 2.3.33 contains [https://github.com/apache/freemarker/pull/104](#104), and not this PR.
See also Jira issue for Jakarta support: [https://issues.apache.org/jira/browse/FREEMARKER-218](FREEMARKER-218)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "JacquesLeRoux (via GitHub)" <gi...@apache.org>.
JacquesLeRoux commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-2027428392
The Apache OFBiz project will finalise its test period next week : https://issues.apache.org/jira/browse/OFBIZ-12935
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [freemarker] ddekany commented on pull request #94: FREEMARKER-218 - Add jakarta.servlet support
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1626879427
Also, unlike with 3.0, 2.4 would use the same packages, so... it's not possible to have both 2.3 and 2.4 in the classpath (except for OSGi). So backward compatibility breaking can be painful for some users. (Also, we probably don't want to support 2.3.x and a 2.4.x line, and less backward compatibility breaking helps there too, like, politically.) So, yeah, keeping BC is possible as far as I know, and it would make the process much easier, as far as I see. Even if, yes, the source code is then more complex... this is a tradeoff.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "anabright (via GitHub)" <gi...@apache.org>.
anabright commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1851879071
Hello, any news about this? I see the mailing list discussion hasn't been very active lately....
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] FREEMARKER-218 - Add jakarta.servlet support [freemarker]
Posted by "ddekany (via GitHub)" <gi...@apache.org>.
ddekany commented on PR #94:
URL: https://github.com/apache/freemarker/pull/94#issuecomment-1871239493
What features of Servlet/JSP support are you using? I'm asking because support JSP taglibs (TLD-based) might brings some complications, if that has to work on modern containers.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@freemarker.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org