Avoid UnsupportedOperationException when setting POSIX permissions on non-POSIX filesystems

Author: farouk-01

gradle-jdks/src/main/java/com/palantir/gradle/jdks/GradleJdksConfigsUtils.java

@@ -100,6 +100,9 @@ public static void writeConfigurationFile(Path pathFile, String content) {
     }
 
     public static void setExecuteFilePermissions(Path path) {
+        if (!path.getFileSystem().supportedFileAttributeViews().contains("posix")) {
+            return;
+        }
         try {
             Set<PosixFilePermission> perms = Files.getPosixFilePermissions(path);
             perms.addAll(Set.of(

gradle-jdks/src/test/groovy/com/palantir/gradle/jdks/PosixPermissionsWindowsTest.groovy

@@ -2,22 +2,33 @@ package com.palantir.gradle.jdks
 
 import spock.lang.Specification
 import spock.lang.TempDir
-
 import java.nio.file.Files
 import java.nio.file.Path
+import java.nio.file.attribute.PosixFilePermission
 
 class PosixPermissionsWindowsTest extends Specification {
 
     @TempDir Path tempDir
 
-    def 'setExecuteFilePermissions crashes on Windows when POSIX is not supported'() {
+    def 'setExecuteFilePermissions works on both POSIX and non-POSIX filesystems'() {
         given:
         Path p = Files.createFile(tempDir.resolve("x.sh"))
+        boolean isPosixSupported = Files.getFileStore(p).supportsFileAttributeView("posix")
 
         when:
         GradleJdksConfigsUtils.setExecuteFilePermissions(p)
 
         then:
-        thrown(RuntimeException) 
+        noExceptionThrown()
+
+        and:
+        if (isPosixSupported) {
+            def perms = Files.getPosixFilePermissions(p)
+            assert perms.contains(PosixFilePermission.OWNER_EXECUTE)
+            assert perms.contains(PosixFilePermission.GROUP_EXECUTE)
+            assert perms.contains(PosixFilePermission.OTHERS_EXECUTE)
+        } else {
+            assert Files.isExecutable(p) || p.toFile().canExecute()
+        }
     }
-}
+}