mbien commented on code in PR #8482: URL: https://github.com/apache/netbeans/pull/8482#discussion_r2095234136
########## .github/workflows/main.yml: ########## @@ -60,9 +60,16 @@ env: # what to build and test, see nbbuild/cluster.properties CLUSTER_CONFIG: 'full' - # default java distribution used by the setup-java action + # default java distribution used by the setup-java action across jobs # see https://github.com/actions/setup-java#supported-distributions - DEFAULT_JAVA_DISTRIBUTION: 'zulu' + # CI requirements: + # - timely GA releases (no delays if possible) + # - EA builds must be available for the upcomming release (note: some vendors skip builds) + # - linux, win, mac (arm) + # - reliable / no extra sauce + DEFAULT_JAVA_DISTRIBUTION: 'temurin' +# DEFAULT_JAVA_DISTRIBUTION: 'sapmachine' +# DEFAULT_JAVA_DISTRIBUTION: 'zulu' Review Comment: I checked again (before merge) just to be sure but zulu is still at build 19 of JDK 25 (https://cdn.azul.com/zulu/bin/). We need at least b22 (current is 23). The other vendors filtered out after running the [brute force availability check](https://gist.github.com/mbien/7911a1272e2bc77234cd7ffca35cdcb4) again. So we could switch to `temurin` or `sapmachine` - nice to have multiple options IMO. -- 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: notifications-unsubscr...@netbeans.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org For additional commands, e-mail: notifications-h...@netbeans.apache.org For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
