-
Notifications
You must be signed in to change notification settings - Fork 103
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Staged EL TCK contains reference to unavailable SNAPSHOT version of parent #1298
Comments
It should be noted that this bug is also present in the official release of the TCK at https://download.eclipse.org/jakartaee/expression-language/6.0/jakarta-expression-language-tck-6.0.0.zip Open Liberty would appreciate this being fixed in a formal 6.0.x release, as it's a silly bug to have to work around to get things running out of the box. |
I have staged https://www.eclipse.org/downloads/download.php?file=/ee4j/jakartaee-tck/jakartaee11/staged/eftl/jakarta-expression-language-tck-6.0.1.zip. Can you verify this bundle once so it can be released to official location. |
Sorry, I missed this comment when it was first posted. I can confirm that the staged 6.0.1 EL TCK resolves this issue to our satisfaction with Open Liberty. Presumably @markt-asf would feel the same way? |
I'm happy if @brideck is happy |
Sorry, scratch that. 6.0.1 appears to be broken. I'm not 100% certain as I can't figure out how the 6.0.1 EL TCK was built. If someone could point me to the tag that was used that would help. I'm not even sure which branch it was created from. What I can tell is that 6.0.1 depends on sigtest-maven-plugin which has changed the syntax for the IgnoreJDKClass option. 6.0.1 still uses the old syntax which means that the signature test fails. Forcing the sigtest-maven-plugin dependency back to 2.2 fixes the problem. |
I will take a look today.
I have a pending change to fix the sig test compatibility with the ignore
option (old syntax should work and new way also).
…On Sat, Jul 20, 2024, 5:37 AM Mark Thomas ***@***.***> wrote:
Sorry, scratch that. 6.0.1 appears to be broken.
I'm not 100% certain as I can't figure out how the 6.0.1 EL TCK was built.
If someone could point me to the tag that was used that would help. I'm not
even sure which branch it was created from.
What I can tell is that 6.0.1 depends on sigtest-maven-plugin which has
changed the syntax for the IgnoreJDKClass option. 6.0.1 still uses the old
syntax which means that the signature test fails. Forcing the
sigtest-maven-plugin dependency back to 2.2 fixes the problem.
—
Reply to this email directly, view it on GitHub
<#1298 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACRJEGALUUITBZQCOOEZH3ZNIVVDAVCNFSM6AAAAABHNBLKFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBRGA3DEMZRG4>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
…igtest 2.3 Signed-off-by: Scott Marlow <smarlow@redhat.com>
I started the https://ci.eclipse.org/jakartaee-tck/job/11/job/standalone-tck/job/eftl-EL-standalonetck-build/20 job which should re-stage the EL 6.0.1. TCK with change 0004e52 |
http://download.eclipse.org/ee4j/jakartaee-tck/jakartaee11/staged/eftl/ has updated https://www.eclipse.org/downloads/download.php?file=/ee4j/jakartaee-tck/jakartaee11/staged/eftl/jakarta-expression-language-tck-6.0.1.zip
Please let me know if the signature tests work now for you. As well as any other problems. I'll push a tag if all tests pass for you. |
Tx for the quick response. All looks good now. EL 6.0.1 TCK passes with current Tomcat 11.0.x source. |
Tag for this staged build is https://github.com/jakartaee/platform-tck/releases/tag/expression-language-tck-6.0.1.zip |
Thanks Scott for the fix! I created PR jakartaee/specifications#768 to release the TCK bundle. |
Not sure if you're waiting for me at all, but it looks like I need to make a quick update on my end to handle the follow-on sigtest update. Should be able to give this the all clear by the end of today. |
All right, that didn't take nearly as long as I thought it might. The more recent staged 6.0.1 release looks good on my end as well. |
Describe the bug
The EL 6.0.0 TCK staged at https://download.eclipse.org/ee4j/jakartaee-tck/jakartaee11/staged/eftl/ contains a reference (under META-INF/maven) to version 10.0.0-SNAPSHOT of its parent POM. There are two issues:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Maven shoudl be able to resolve all of the dependencies of this JAR without using snapshots.
Additional context
It can be worked around by changing the reference to jakarta.tck:project:11.0.0-M1 in the pom file generated for this JAR in your local Maven repository.
The text was updated successfully, but these errors were encountered: