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
Replace deprecated core.launcher.Main by equinox.launcher.Main #1266
Replace deprecated core.launcher.Main by equinox.launcher.Main #1266
Conversation
Test Results 617 files ±0 617 suites ±0 43m 41s ⏱️ + 6m 9s For more details on these failures and errors, see this check. Results for commit efc1ff8. ± Comparison against base commit 072423e. |
The test-failures seem to be the same as on the master, but I assume they are caused by eclipse-platform/eclipse.platform.releng.aggregator#1919 (comment), aren't they? |
Probably, yes. The same failures occur in other PRs, such as #852. I've retriggered the GH actions for #852 this morning, when yesterday's I-Build had been deployed, but the same failures still occur. |
In https://github.com/eclipse-platform/eclipse.platform/actions/runs/8426539577/job/23092286324?pr=1266 I can see it's some test failure in ant.tests.ui but can't figure more from the log. Can someone help finding the details? |
As Jenkins succeeds it's fine to merge. |
Thanks Heiko.
Me too and since the deprecated class just wraps the replacement in the same bundle I think it is relatively save. When I used it locally in my IDE it worked. |
I did not have time to investigate yet, but I could imagine that this is somehow related to using a Java 21 JVM. I also face and currently investigate issues in another PR, in which a test is working fine when run with a Java 17 JVM but does not succeed with a Java 21 one: eclipse-platform/eclipse.platform.ui#1775 |
I just verified: Running the Ant UI tests with a Java 21 JVM locally produces the same test failures as the GH actions builds here. |
I wonder if |
https://ci.eclipse.org/platform/job/eclipse.platform/job/master/768/ is finally a good build with even comparator issues fixed. |
Yes, that makes the Ant UI tests run properly. |
Required for eclipse-equinox/equinox#562