Skip to content

Fix BZ 70001: _jspx_dependants contains non-deterministic absolute JA…#992

Merged
markt-asf merged 1 commit intoapache:mainfrom
jluehe:taglib
Apr 29, 2026
Merged

Fix BZ 70001: _jspx_dependants contains non-deterministic absolute JA…#992
markt-asf merged 1 commit intoapache:mainfrom
jluehe:taglib

Conversation

@jluehe
Copy link
Copy Markdown
Contributor

@jluehe jluehe commented Apr 14, 2026

…R paths for taglibs resolved from external JARs

…R paths for taglibs resolved from external JARs
@rmaucher
Copy link
Copy Markdown
Contributor

It looks good to me. The situation it covers is not supposed to be the main usual case obviously (never been a fan of the "external shared jar" use case, but I also got the comments "I have 1000 webapps" in the past, so ...) but it's a nice improvement. Let's see if there are any final objections to this.

@jluehe
Copy link
Copy Markdown
Contributor Author

jluehe commented Apr 24, 2026

It looks good to me. The situation it covers is not supposed to be the main usual case obviously (never been a fan of the "external shared jar" use case, but I also got the comments "I have 1000 webapps" in the past, so ...) but it's a nice improvement. Let's see if there are any final objections to this.

Thank you, @rmaucher!

@jluehe
Copy link
Copy Markdown
Contributor Author

jluehe commented Apr 28, 2026

Hi @rmaucher, just circling back on this to see if it can be merged and included in the next release - or would you rather wait a bit longer? Thank you!

@markt-asf
Copy link
Copy Markdown
Contributor

LGTM. I'll merge, run the TCKs tests and back-port assuming there are no issues.

@markt-asf markt-asf merged commit 6e16bd9 into apache:main Apr 29, 2026
6 checks passed
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