Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-30 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2842796053 Thank you, Uwe. -- 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 u

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-30 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2841596506 One information about Exclipse: With the recent commit all works fine. It just sometimes fails with a compilation error (but that's not new) regarding: "Permitted type LeafReaderConte

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-30 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2841596170 Fixed: 0550cc6d2d0294641679131b597d591896211fce -- 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

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-30 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2841595622 I got a strange error: * What went wrong: Execution failed for task ':downloadEcjDrop'. > A failure occurred while executing de.undercouch.gradle.tasks.download.internal.

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-30 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2841274438 > I think we need to change the gradlew / gradlew.cmd files to pass the additional command line option `--enable-native-access=ALL-UNNAMED` gradle is now launched with --jar and con

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840152208 I also updated the 10.x release tester job to use Java 24, too. This will affect the next release. Of course Gradle backport is needed. -- This is an automated message from the Apac

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840142555 Policeman Jenkins was also updated. I hope the randomization and RUNTIME_JAVA_HOME with 25 works, too. If something breaks, I will keep an eye on it. -- This is an automated messag

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840100565 > > I think we need to change the gradlew / gradlew.cmd files to pass the additional command line option --enable-native-access=ALL-UNNAMED > > I didn't touch the launcher scrip

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840085005 > I disabled Policeman Jenkins Main Jobs, working on Upgrade. Can you take care of ASF Jenkins? I've switched all jobs on asf jenkins to Java 24. Hopefully didn't miss anything.

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840083351 > I think we need to change the gradlew / gradlew.cmd files to pass the additional command line option --enable-native-access=ALL-UNNAMED I didn't touch the launcher scripts... coul

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840073232 I updated the other PR and generated the APIJAR for 24. For the next update, basically the same linke this commit needs to be done: https://github.com/apache/lucene/pull/14564/commits

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840040245 When testing this I figured out the following warning when starting with Java 24: ``` WARNING: A restricted method in java.lang.System has been called WARNING: java.lang.S

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2840008944 I disabled Policeman Jenkins Main Jobs, working on Upgrade. Can you take care of ASF Jenkins? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2839969476 Ok, I've applied this to main. I will backport gradle upgrade selectively to 10x tomorrow (can't be cherry picked automatically). -- This is an automated message from the Apache Git Ser

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss merged PR #14533: URL: https://github.com/apache/lucene/pull/14533 -- 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: issues-unsubscr...@lucene.apac

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2839972055 Ok, I've applied this to main. I will backport gradle upgrade selectively to 10x tomorrow (can't be cherry picked automatically). -- This is an automated message from the Apache Git Ser

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2839970921 Ok, I've applied this to main. I will backport gradle upgrade selectively to 10x tomorrow (can't be cherry picked automatically). -- This is an automated message from the Apache Git Ser

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2839968972 Ok, I've applied this to main. I will backport gradle upgrade selectively to 10x tomorrow (can't be cherry picked automatically). -- This is an automated message from the Apache Git Ser

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on PR #14533: URL: https://github.com/apache/lucene/pull/14533#issuecomment-2839315056 @dweiss: I opened https://github.com/apache/lucene/pull/14564 to get rid of `MMapDirexctoy` abstractions. I can wait with merging this until this one is merged. The update to Java 24

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on code in PR #14533: URL: https://github.com/apache/lucene/pull/14533#discussion_r2066466992 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -365,7 +365,9 @@ private static MMapIndexInputProvider lookupProvider() { // acce

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
uschindler commented on code in PR #14533: URL: https://github.com/apache/lucene/pull/14533#discussion_r2066461653 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -365,7 +365,9 @@ private static MMapIndexInputProvider lookupProvider() { // acce

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on code in PR #14533: URL: https://github.com/apache/lucene/pull/14533#discussion_r2066149285 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -365,7 +365,9 @@ private static MMapIndexInputProvider lookupProvider() { // access t

Re: [PR] Upgrade to Gradle 8.14 and Java 24 minimum [lucene]

2025-04-29 Thread via GitHub
dweiss commented on code in PR #14533: URL: https://github.com/apache/lucene/pull/14533#discussion_r2066138666 ## lucene/core/src/java/org/apache/lucene/store/MMapDirectory.java: ## @@ -365,7 +365,9 @@ private static MMapIndexInputProvider lookupProvider() { // access t