Draft: Try updating xbean-asm#199
Draft: Try updating xbean-asm#199jgallimore wants to merge 7 commits intotomitribe:tomee-1.7.x-TT.x-releasefrom
Conversation
pom.xml
Outdated
| <groupId>commons-pool</groupId> | ||
| <artifactId>commons-pool</artifactId> | ||
| </exclusion> | ||
| <exclusion> |
There was a problem hiding this comment.
Unfortunately, these shaded jars change the relocated package so usages of org.apache.xbean.asm5 become org.apache.xbean.asm9 in the code. The version of OpenWebBeans in use in this version of TomEE still uses ASM5, so we'd either need to patch that version of OWB to use ASM9, or continue to use both (assuming that works).
There was a problem hiding this comment.
The version of OpenWebBeans in use in this version of TomEE still uses ASM5, so we'd either need to patch that version of OWB to use ASM9, or continue to use both (assuming that works).
yesterday I executed the test locally on one open-ejb related module from this PR and test passes, I guess I was lucky so I decided to move with CI for getting the full picture. But yes, the package in OWB changed :( .
|
test this please. |
|
test this please. |
|
We were getting inconsistencies between CI resuls for 1.7.6-TT.28 and this PR because the PR was based on OS 1.7.6 and not on 1.7.6-TT.28. The result for c65d683 would be the ones that we can compare between Before and After CI results. |
|
test this please. |
|
This PR gest +25 failures than tomee-1.7.x-TT.x-release branch. I see errors like: So open-jpa 2.4.3 also brings xbean-asm6-shaded jars So based on the above it seems we need to patch open-jpa 2.4.3 to use asm9 instead of asm6, release it and test again this PR bu using open-jpa 2.4.3-T.1. |
This attempts to upgrade the version of xbean-asm to a recent version to allow jars with module-info to be scanned. This version of TomEE is old, and so OWB still brings in an older version of xbean-asm, so jars with module-info may still fail.