Changes to support modern gradle

build.gradle

@@ -21,7 +21,6 @@ buildscript {
 plugins {
     id 'java'
     id 'war'
-    id 'eclipse'
     id 'idea'
     id 'org.sonarqube' version '6.2.0.5505'
 }
@@ -32,15 +31,17 @@ plugins {
 //  jhrg 1/22/24
 sonarqube {
     properties {
+        property "sonar.gradle.skipCompile", "true"
+
         property "sonar.sourceEncoding", "UTF-8"
         property "sonar.projectKey", "opendap-olfs"
         property "sonar.projectName", "olfs"
         property "sonar.projectVersion", "${OLFS_VERSION}"
         property "sonar.organization", "opendap"
         property "sonar.inclusions", "src"
         property "sonar.exclusions", "src/opendap/Coverity_Model.java," +
-                                    "retired/**/*.java," +
-                                    "src/opendap/**/*Test.java"
+                "retired/**/*.java," +
+                "src/opendap/**/*Test.java"
         property "sonar.java.binaries", "build"
         property "sonar.java.libraries", "lib/*.jar"
     }
@@ -58,8 +59,13 @@ if (hasProperty('buildScan')) {
 group = 'org.opendap'
 description = 'The OLFs build'
 
-sourceCompatibility = 1.9
-targetCompatibility = 1.9
+java {
+    sourceCompatibility = JavaVersion.VERSION_1_9
+    targetCompatibility = JavaVersion.VERSION_1_9
+}
+
+// sourceCompatibility = 1.8
+// targetCompatibility = 1.8
 
 repositories {
     // For just about everything except...
@@ -304,40 +310,40 @@ dependencies {
 // Substitute the version information before compilation
 compileJava {
     doFirst {
-    	println 'HYRAX_VERSION: ' + project.HYRAX_VERSION
+        println 'HYRAX_VERSION: ' + project.HYRAX_VERSION
     }
 
     dependsOn 'substituteVersionInfo'
 }
 
 // see https://docs.gradle.org/current/dsl/org.gradle.api.plugins.ExtraPropertiesExtension.html
 project.ext {
-    resources_dir = "${projectDir}/resources"
+    resources_dir = "${projectDir}/resources" as GStringImpl
     // WebInf_resources_dir = "${project.resources_dir}/hyrax/WEB-INF"
     // distribution_resources_dir = "${project.resources_dir}/distribution"
 
     // Build Directories
-    build_dir = "${buildDir}"
-    build_classes = "${project.build_dir}/classes"
-    build_docs = "${project.build_dir}/docs"
-    build_dist = "${project.build_dir}/dist"
-    build_lib = "${project.build_dir}/lib"
-    build_run = "${project.build_dir}/run"
-    build_src = "${project.build_dir}/src"
-    build_resources = "${project.build_dir}/resources"
-
-    hyrax_resources_dir = "${project.resources_dir}/hyrax"
-    ngap_resources_dir = "${project.resources_dir}/ngap"
-    robots_resources_dir = "${project.resources_dir}/robots"
-    build_robots_resources = "${project.build_dir}/robots"
-    wcs_resources_dir = "${project.resources_dir}/WCS/2.0"
+    build_dir = "${buildDir}" as GStringImpl
+    build_classes = "${project.build_dir}/classes" as GStringImpl
+    build_docs = "${project.build_dir}/docs" as GStringImpl
+    build_dist = "${project.build_dir}/dist" as GStringImpl
+    build_lib = "${project.build_dir}/lib" as GStringImpl
+    build_run = "${project.build_dir}/run" as GStringImpl
+    build_src = "${project.build_dir}/src" as GStringImpl
+    build_resources = "${project.build_dir}/resources" as GStringImpl
+
+    hyrax_resources_dir = "${project.resources_dir}/hyrax" as GStringImpl
+    ngap_resources_dir = "${project.resources_dir}/ngap" as GStringImpl
+    robots_resources_dir = "${project.resources_dir}/robots" as GStringImpl
+    build_robots_resources = "${project.build_dir}/robots" as GStringImpl
+    wcs_resources_dir = "${project.resources_dir}/WCS/2.0" as GStringImpl
 }
 
 clean.doFirst {
     delete "src_gradle"
 }
 
-task init_dir {
+tasks.register('init_dir') {
     description "Create required folders in the build directory."
     doLast {
         mkdir "${project.build_dir}"
@@ -349,7 +355,8 @@ task init_dir {
 // See: https://docs.gradle.org/current/userguide/war_plugin.html
 // We want to make multiple war files (opendap, ROOT, ngap) I think. This answer shows how:
 // given that https://stackoverflow.com/questions/13077694/create-multiple-war-files-with-different-dependencies-in-gradle
-war {
+war
+{
     archiveBaseName = 'opendap'
 
     // Copy/filter the various resource files
@@ -361,7 +368,8 @@ war {
     from layout.buildDirectory.dir("resources")
 }
 
-task buildOpendapWar(type: War, dependsOn: classes){
+tasks.register('buildOpendapWar', War) {
+    dependsOn classes
     archiveBaseName = 'opendap_server'
     destinationDirectory = file("$buildDir/dist")
 
@@ -378,7 +386,8 @@ task buildOpendapWar(type: War, dependsOn: classes){
     from layout.buildDirectory.dir("resources")
 }
 
-task buildNgapWar(type: War, dependsOn: classes){
+tasks.register('buildNgapWar', War) {
+    dependsOn classes
     dependsOn configurations.ngapDep
     archiveBaseName = 'opendap_ngap'
     destinationDirectory = file("$buildDir/dist")
@@ -396,25 +405,29 @@ task buildNgapWar(type: War, dependsOn: classes){
     from layout.buildDirectory.dir("resources")
 }
 
-task buildRobotsWar(type: War, dependsOn: classes){
+tasks.register('buildRobotsWar', War) {
+    dependsOn classes
     archiveBaseName = 'opendap_robots'
     destinationDirectory = file("$buildDir/dist")
 
     group = 'warBuilds'
     description = 'Build the Opendap Robots war.'
 }
 
-task buildDists(dependsOn: [buildOpendapWar, buildNgapWar]){ //hold for later ", buildRobotsWar"
+/*
+tasks.register('buildDists') {
+    dependsOn[buildOpendapWar, buildNgapWar] //hold for later ", buildRobotsWar"
     group = 'warBuilds'
     description = 'Build the Opendap, Ngap, and Robots war.'
 }
+*/
 
 clean {
     // FIXME: Awful. instead, look for all (.+)\\.java.in files and delete $1.java jhrg 9/16/21
-    delete 'src/opendap/bes/Version.java'
+    // delete 'src/opendap/bes/Version.java' // I had to replace Version.java, we need it. ndp 07/18/24
 }
 
-tasks.withType(JavaCompile) {
+tasks.withType(JavaCompile).configureEach {
     //options.compilerArgs << '-Xlint:unchecked'
     //options.compilerArgs << '-Xlint:unchecked' << '-Werror' //enable to mark deprecated warnings as errors
     options.deprecation = true
@@ -449,10 +462,11 @@ tasks.withType(JavaCompile) {
 // jhrg 9/15/21
 
 import org.apache.tools.ant.filters.ReplaceTokens
+import org.codehaus.groovy.runtime.GStringImpl
 
-def tokens = [SERVICE_CONTEXT: project.DEPLOYMENT_CONTEXT,
-              HyraxVersion: project.HYRAX_VERSION,
-              OlfsVersion: project.OLFS_VERSION,
+def tokens = [SERVICE_CONTEXT   : project.DEPLOYMENT_CONTEXT,
+              HyraxVersion      : project.HYRAX_VERSION,
+              OlfsVersion       : project.OLFS_VERSION,
               WcsSoftwareVersion: project.WCS_VERSION]
 
 // Not used. jhrg 9/17/21 WCS_SOFTWARE_VERSION: project.WCS_VERSION
@@ -517,7 +531,7 @@ tasks.register('copyWCSResources', Copy) {
 // '.java'.
 //
 // I made this to see if I could and because I wanted to make the files that get
-// tokens substituted explict in the sources. There might be better ways to do this.
+// tokens substituted explicit in the sources. There might be better ways to do this.
 // And handling the substitution this way makes the clean target more complex.
 // jhrg 9/16 21
 tasks.register('substituteVersionInfo', Copy) {

gradle.properties

@@ -4,3 +4,4 @@
 systemProp.https.protocols = TLSv1.2
 
 systemProp.sonar.host.url=https://sonarcloud.io
+systemProp.sonar.gradle.skipCompile=true

src/opendap/bes/Version.java

@@ -44,18 +44,11 @@ public class Version  {
     private static final String hyraxVersion = "@HyraxVersion@";
 
 
-    /**
-     * Returns a String containing the OLFS version.
-     * @return The version of OLFS.
-     */
-    public static String getOLFSVersionString() {
-        return (olfsVersion);
-    }
-
     /**
      * Returns a String containing the Hyrax version.
      * @return The version of Hyrax.
-     */    public static String getHyraxVersionString() {
+     */
+    public static String getHyraxVersionString() {
         return (hyraxVersion);
     }
 
@@ -80,22 +73,12 @@ public static Element getHyraxVersionElement() {
         return (hyrax);
     }
 
-    /**
-     *  Produce the ServerUUID value used by the top level of Hyrax.
-     * @return The UUID.
-     */
-    public static String getServerUUID(){
-        return "e93c3d09-a5d9-49a0-a912-a0ca16430b91";
-    }
-
-
 
     /**
      *
      * @param request The client request for which to return the verison.
      * @return A string containing the value of the XDODS-Server MIME header as
      * ascertained by querying the BES.
-     * @throws Exception If these is a problem getting the version document.
      */
     public static String getXDODSServerVersion(HttpServletRequest request) {
         String relativeUrl = ReqInfo.getLocalUrl(request);
@@ -109,7 +92,11 @@ public static String getXDODSServerVersion(HttpServletRequest request) {
      * @param request The client request for which to return the verison.
      * @return A String containing the value of the XOPeNDAP-Server MIME header ascertained by querying
      *         the BES and conforming to the DAP4 specification.
-     * @throws Exception If these is a problem getting the version document.
+     * @throws JDOMException If the BES response cannot be parsed.
+     * @throws BadConfigurationException If the configuration doc has incorrect/missing/unexpected content.
+     * @throws PPTException If there is a problem communicating with the BES.
+     * @throws BESError If there is a problem communicating with the BES.
+     * @throws IOException If there is a problem communicating with the BES.
      */
     public static String getXOPeNDAPServerVersion(HttpServletRequest request)
             throws JDOMException, BadConfigurationException, PPTException, BESError, IOException {
@@ -139,7 +126,6 @@ public static String getXOPeNDAPServerVersion(HttpServletRequest request)
      * @param request The client request.
      * @return A String containing the XDAP MIME header value that describes
      * the DAP specifcation that the server response conforms to.
-     * @throws Exception If these is a problem getting the version document.
      */
     public static String getXDAPVersion(HttpServletRequest request) {
         String responseDAP = null;
@@ -174,7 +160,6 @@ public static String getXDAPVersion(HttpServletRequest request) {
      *
      * @param request Client request to serviced
      * @param response The response in which to set the headers.
-     * @throws Exception If these is a problem getting the version document.
      */
     public static void setOpendapMimeHeaders(HttpServletRequest request, HttpServletResponse response)
             throws JDOMException, BadConfigurationException, PPTException, IOException, BESError {