You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by re...@apache.org on 2018/02/01 10:45:01 UTC
svn commit: r1822863 -
/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java
Author: reschke
Date: Thu Feb 1 10:45:01 2018
New Revision: 1822863
URL: http://svn.apache.org/viewvc?rev=1822863&view=rev
Log:
JCR-4253: RepositoryConfig: add some handling for mkdir failure
(patch by diego.schivo - thanks)
Modified:
jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java
Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java?rev=1822863&r1=1822862&r2=1822863&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/config/RepositoryConfig.java Thu Feb 1 10:45:01 2018
@@ -192,10 +192,13 @@ public class RepositoryConfig
private static void installRepositorySkeleton(
File dir, File xml, URL resource)
- throws IOException {
+ throws IOException, ConfigurationException {
if (!dir.exists()) {
log.info("Creating repository directory {}", dir);
- dir.mkdirs();
+ boolean dirCreated = dir.mkdirs();
+ if (!dirCreated) {
+ throw new ConfigurationException("Cannot create repository directory " + dir);
+ }
}
if (!xml.exists()) {
@@ -541,7 +544,10 @@ public class RepositoryConfig
// Get the physical workspace root directory (create it if not found)
File directory = new File(workspaceDirectory);
if (!directory.exists()) {
- directory.mkdirs();
+ boolean directoryCreated = directory.mkdirs();
+ if (!directoryCreated) {
+ throw new ConfigurationException("Cannot create workspace root directory " + directory);
+ }
}
// Get all workspace subdirectories