JAL-3322 corrected fix to convertBuildingMD task fail. removed debugging warnings...
authorBen Soares <bsoares@dundee.ac.uk>
Fri, 28 Jun 2019 09:33:18 +0000 (10:33 +0100)
committerBen Soares <bsoares@dundee.ac.uk>
Fri, 28 Jun 2019 09:33:18 +0000 (10:33 +0100)
build.gradle
getdown/lib/getdown-core.jar
getdown/lib/getdown-launcher.jar
getdown/src/getdown/ant/pom.xml
getdown/src/getdown/core/pom.xml
getdown/src/getdown/core/src/main/java/com/threerings/getdown/data/Application.java
getdown/src/getdown/launcher/pom.xml
getdown/src/getdown/pom.xml
j11lib/getdown-core.jar
j8lib/getdown-core.jar

index 165c05e..1f16133 100644 (file)
@@ -595,7 +595,7 @@ task convertBuildingMD(type: Exec) {
   }
 
   doFirst {
-    if (false && pandoc != null && file(pandoc).exists()) {
+    if (pandoc != null && file(pandoc).exists()) {
         commandLine pandoc, '-s', '-o', buildingHTML, '--metadata', 'pagetitle="Building Jalview from Source"', '--toc', '-H', css, buildingMD
     } else {
         println("Cannot find pandoc. Skipping convert building.md to HTML")
index ff2d326..ea204c6 100644 (file)
Binary files a/getdown/lib/getdown-core.jar and b/getdown/lib/getdown-core.jar differ
index 8d8a2b3..10072c9 100644 (file)
Binary files a/getdown/lib/getdown-launcher.jar and b/getdown/lib/getdown-launcher.jar differ
index 22b9995..7f35d44 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>com.threerings.getdown</groupId>
     <artifactId>getdown</artifactId>
-    <version>1.8.3-1.1_JVL</version>
+    <version>1.8.3-1.1.2_JVL</version>
   </parent>
 
   <artifactId>getdown-ant</artifactId>
index 6fff245..2350118 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>com.threerings.getdown</groupId>
     <artifactId>getdown</artifactId>
-    <version>1.8.3-1.1_JVL</version>
+    <version>1.8.3-1.1.2_JVL</version>
   </parent>
 
   <artifactId>getdown-core</artifactId>
index 0ba2a5e..5c61378 100644 (file)
@@ -282,16 +282,17 @@ public class Application
      *
      */
     public Application (EnvConfig envc) {
-        _envc = envc;
-        _config = getLocalPath(envc.appDir, CONFIG_FILE);
-        _backupConfig = getLocalPath(new File(envc.appDir.getAbsolutePath()+File.separator+BACKUP_CONFIG_DIR), CONFIG_FILE);
+       _envc = envc;
+       _config = getLocalPath(envc.appDir, CONFIG_FILE);
+       _backupConfig = getLocalPath(envc.appDir, BACKUP_CONFIG_DIR+File.separator+CONFIG_FILE);
+       log.warning("Backup config file now", "_backupConfig", _backupConfig, "exists", _backupConfig.exists(), "isReadable", _backupConfig.canRead());
     }
 
     /**
      * Returns the configured application directory.
      */
     public File getAppDir () {
-        return _envc.appDir;
+       return _envc.appDir;
     }
 
     /**
@@ -300,7 +301,7 @@ public class Application
      */
     public boolean useCodeCache ()
     {
-        return _useCodeCache;
+       return _useCodeCache;
     }
 
     /**
@@ -309,7 +310,7 @@ public class Application
      */
     public int getCodeCacheRetentionDays ()
     {
-        return _codeCacheRetentionDays;
+       return _codeCacheRetentionDays;
     }
 
     /**
@@ -317,7 +318,7 @@ public class Application
      * app files from its hosting server.
      */
     public int maxConcurrentDownloads () {
-        return _maxConcDownloads;
+       return _maxConcDownloads;
     }
 
     /**
@@ -326,21 +327,11 @@ public class Application
     public Resource getConfigResource ()
     {
        Resource res = null;
-        try {
-            res = createResource(CONFIG_FILE, Resource.NORMAL);
-        } catch (Exception e) {
-            log.warning("Invalid appbase '" + _vappbase + "'.", e);
-        }
-        if (res == null) {
-               try {
-                               String backupConfig = BACKUP_CONFIG_DIR + File.separator + CONFIG_FILE;
-                       log.warning("Could not read config '"+CONFIG_FILE+"'. Attempting to use backup '"+backupConfig+"'");
-                               res = createResource(backupConfig, Resource.NORMAL);
-               } catch (Exception ex) {
-                               throw new RuntimeException("Invalid appbase in backup config'" + _vappbase + "'.", ex);
-               }
-        }
-        return res;
+       try {
+               return createResource(CONFIG_FILE, Resource.NORMAL);
+       } catch (Exception e) {
+               log.warning("Invalid appbase '" + _vappbase + "'.", e);
+       }
     }
 
     /**
@@ -348,7 +339,7 @@ public class Application
      */
     public List<Resource> getCodeResources ()
     {
-        return _codes;
+       return _codes;
     }
 
     /**
@@ -617,7 +608,8 @@ public class Application
         }
         if (config == null || config.getString("appbase") == null || config.getString("appbase").isEmpty()) {
                try {
-                       config = Config.parseConfig(_backupConfig, opts);
+                       Config backupConfig = Config.parseConfig(_backupConfig, opts);
+                       config = backupConfig;
                        log.warning("Using backup config file", "appdir", getAppDir(), "backupConfig", _backupConfig.getAbsoluteFile());
                } catch (Exception e) {
                        log.warning("Failure reading backup config file", "file", _backupConfig, e);
index 1a35bee..b809235 100644 (file)
@@ -4,7 +4,7 @@
   <parent>
     <groupId>com.threerings.getdown</groupId>
     <artifactId>getdown</artifactId>
-    <version>1.8.3-1.1_JVL</version>
+    <version>1.8.3-1.1.2_JVL</version>
   </parent>
 
   <artifactId>getdown-launcher</artifactId>
index f11f925..64555e0 100644 (file)
@@ -10,7 +10,7 @@
   <groupId>com.threerings.getdown</groupId>
   <artifactId>getdown</artifactId>
   <packaging>pom</packaging>
-  <version>1.8.3-1.1_JVL</version>
+  <version>1.8.3-1.1.2_JVL</version>
 
   <name>getdown</name>
   <description>An application installer and updater.</description>
index ff2d326..ea204c6 100644 (file)
Binary files a/j11lib/getdown-core.jar and b/j11lib/getdown-core.jar differ
index ff2d326..ea204c6 100644 (file)
Binary files a/j8lib/getdown-core.jar and b/j8lib/getdown-core.jar differ