leonfin commented on PR #7930: URL: https://github.com/apache/geode/pull/7930#issuecomment-3312742997
I've built with java 17 with tests and no issues observed. Just want to mention, I use Intellij and I do run those steps in BUILDING.md and it works for me on 2025.2.1. Only 2 things that don't work (never worked before either, not related to Java 17 changes) 1. RepeatTestTest from geode-repeat-test module. I always have to manually change package imports there to replace junit 4.x with: `import org.junit.jupiter.api.Test import static org.junit.jupiter.api.Assertions.assertTrue` 2. For geode-annotation-processor module I have to update build.gradle with ` testImplementation 'org.junit.vintage:junit-vintage-engine:5.8.1' implementation 'junit:junit:4.13.1' testImplementation 'junit:junit:4.13.1' ` Otherwise it fails in EnsureCorrectRunsWithProcessor class which uses junit4. Not sure how it works from command line gradle build. But it clearly uses junit4 but doesn't import it in build.gradle -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
