geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r560020 - /geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
Date Thu, 26 Jul 2007 22:28:16 GMT
Author: djencks
Date: Thu Jul 26 15:28:15 2007
New Revision: 560020

URL: http://svn.apache.org/viewvc?view=rev&rev=560020
Log:
GERONIMO-3358 local attribute manager should not require write access to config dir if it's
read-only

Modified:
    geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java

Modified: geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java?view=diff&rev=560020&r1=560019&r2=560020
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
(original)
+++ geronimo/server/branches/2.0/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/LocalAttributeManager.java
Thu Jul 26 15:28:15 2007
@@ -583,7 +583,10 @@
                 throw new IOException("Unable to create directory for list:" + parent);
             }
         }
-        if (!parent.canRead() || !parent.canWrite()) {
+        if (!parent.canRead()) {
+            throw new IOException("Unable to read manageable attribute files in directory
" + parent.getAbsolutePath());
+        }
+        if (!readOnly && !parent.canWrite()) {
             throw new IOException("Unable to write manageable attribute files to directory
" + parent.getAbsolutePath());
         }
     }



Mime
View raw message