Skip to content

Support Legacy and Jakarta-Prefixed Resource Files in JAF API with Legacy Precedence #186#187

Open
jbescos wants to merge 1 commit intojakartaee:mainfrom
jbescos:issue186
Open

Support Legacy and Jakarta-Prefixed Resource Files in JAF API with Legacy Precedence #186#187
jbescos wants to merge 1 commit intojakartaee:mainfrom
jbescos:issue186

Conversation

@jbescos
Copy link
Copy Markdown
Member

@jbescos jbescos commented Oct 2, 2025

#186

In this diff is where I added the jakarta resources. It could help to review this PR.

Tested in angus-activation here: eclipse-ee4j/angus-activation#60

@jbescos jbescos marked this pull request as draft October 2, 2025 08:18
@jbescos jbescos marked this pull request as ready for review October 2, 2025 09:42
…gacy Precedence jakartaee#186

Signed-off-by: Jorge Bescos Gascon <jorge.bescos.gascon@oracle.com>
@jmehrens
Copy link
Copy Markdown
Contributor

jmehrens commented Oct 2, 2025

Sidenote, this should fix WildFly issue but I wonder if mixing javax.mail and JakartaMail end up poisoning each other. The goal of name change was to stop that. I have and alternative patch to fix that and I'll see if I can draft that up.

@jbescos
Copy link
Copy Markdown
Member Author

jbescos commented Oct 3, 2025

Sidenote, this should fix WildFly issue but I wonder if mixing javax.mail and JakartaMail end up poisoning each other. The goal of name change was to stop that. I have and alternative patch to fix that and I'll see if I can draft that up.

Actually you are right, and this is very tricky to completely solve it because there are many different combinations between activation-api, mail-api, angus-activation and angus-mail.

This PR is only a middle step to transition to the new jakarta resources.

@jbescos jbescos marked this pull request as draft October 3, 2025 05:24
@jbescos jbescos marked this pull request as ready for review October 3, 2025 08:38
@lukasj
Copy link
Copy Markdown
Contributor

lukasj commented Mar 27, 2026

what is missing here? Is this for 2.1.x and some other, possibly breaking solution, is for master/2.2?

@jbescos
Copy link
Copy Markdown
Member Author

jbescos commented Apr 13, 2026

I need to revisit this when I have time because I forgot the details. Lets keep it open for the time being.

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

Successfully merging this pull request may close these issues.

3 participants