fixing gradle issue (#735)

* fixing gradle issue

Signed-off-by: Dhrubo Saha <[email protected]>

* fixing CVE-2022-1471

Signed-off-by: Dhrubo Saha <[email protected]>

* fixing CVE issue

Signed-off-by: Dhrubo Saha <[email protected]>

* avoiding deprecation

Signed-off-by: Dhrubo Saha <[email protected]>

---------

Signed-off-by: Dhrubo Saha <[email protected]>

Author: dhrubo-os

client/build.gradle

@@ -22,8 +22,9 @@ dependencies {
 
 jacocoTestReport {
     reports {
-        xml.enabled false
-        csv.enabled false
+        xml.getRequired().set(true)
+        csv.getRequired().set(false)
+        html.getRequired().set(true)
     }
 
     dependsOn test

ml-algorithms/build.gradle

@@ -1,3 +1,5 @@
+import org.gradle.nativeplatform.platform.internal.DefaultNativePlatform
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
@@ -40,7 +42,17 @@ dependencies {
     implementation("ai.djl.onnxruntime:onnxruntime-engine:0.19.0") {
         exclude group: "com.microsoft.onnxruntime", module: "onnxruntime"
     }
-    implementation "com.microsoft.onnxruntime:onnxruntime_gpu:1.13.1"
+    def os = DefaultNativePlatform.currentOperatingSystem
+    //mac doesn't support GPU
+    if (os.macOsX) {
+        dependencies {
+            implementation "com.microsoft.onnxruntime:onnxruntime:1.13.1"
+        }
+    } else {
+        dependencies {
+            implementation "com.microsoft.onnxruntime:onnxruntime_gpu:1.13.1"
+        }
+    }
 }
 
 configurations.all {
@@ -49,8 +61,9 @@ configurations.all {
 
 jacocoTestReport {
     reports {
-        xml.enabled false
-        csv.enabled false
+        xml.getRequired().set(true)
+        csv.getRequired().set(false)
+        html.getRequired().set(true)
     }
 
     dependsOn test

plugin/build.gradle

@@ -229,9 +229,9 @@ task release(type: Copy, group: 'build') {
 
 jacocoTestReport {
     reports {
-        xml.enabled true
-        html.enabled true
-        csv.enabled false
+        xml.getRequired().set(true)
+        csv.getRequired().set(false)
+        html.getRequired().set(true)
     }
 
     dependsOn test
@@ -377,7 +377,7 @@ tasks.withType(licenseHeaders.class) {
 }
 
 checkstyle {
-    toolVersion = '8.29'
+    toolVersion = '8.38'
 }
 
 String bwcVersion = "2.4.0.0"