Merge pull request #1 from paulmer/fix_log4j_warning

Fixed "No appenders could be found" warning by passing a URL for the log...
harvard-lts · Feb 23, 2015 · 67c8453 · 67c8453
2 parents 136854c + b64c9e2
commit 67c8453
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/fits.jar b/lib/fits.jar
diff --git a/src/edu/harvard/hul/ois/fits/Fits.java b/src/edu/harvard/hul/ois/fits/Fits.java
@@ -122,11 +122,9 @@ public Fits( String fits_home ) throws FitsConfigurationException {
     // Now using an explicit properties file, because otherwoise DROID will
     // hijack it, and it's cleaner this way anyway.
 
-    //(SM 1/2/14 -- Note that this statement probably isn't doing what the author intended.
-    //  If log4j.debug=true is set then it shows that this doesn't actually find the specified
-    //  log4j.properties file.  Leaving as is for now since overall logging works as intended.
-    //  also note that any logging statements in this class probably do not work.
-    System.setProperty( "log4j.configuration", FITS_TOOLS + "log4j.properties" );
+    // Construct a URI to the log4j.properties file.
+    File	log4jProperties = new File(FITS_TOOLS + "log4j.properties");
+    System.setProperty( "log4j.configuration", log4jProperties.toURI().toString());
 
     logger = Logger.getLogger( this.getClass() );
     try {