From a12b2b1a0ef15681437540db818dbe396cfb3c25 Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Tue, 13 Mar 2012 08:50:59 +0100
Subject: [PATCH] remove empty directory after failed repository creation

---
 .../AbstractSimpleRepositoryHandler.java      | 29 +++++++++++++++++--
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java b/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
index 9d29db8a87..f453cf2539 100644
--- a/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
+++ b/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
@@ -35,6 +35,8 @@ package sonia.scm.repository;
 
 //~--- non-JDK imports --------------------------------------------------------
 
+import com.google.common.base.Throwables;
+
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -110,9 +112,30 @@ public abstract class AbstractSimpleRepositoryHandler<C extends SimpleRepository
     }
 
     checkPath(directory);
-    fileSystem.create(directory);
-    create(repository, directory);
-    postCreate(repository, directory);
+
+    try
+    {
+      fileSystem.create(directory);
+      create(repository, directory);
+      postCreate(repository, directory);
+    }
+    catch (Exception ex)
+    {
+      if (directory.exists())
+      {
+        if (logger.isDebugEnabled())
+        {
+          logger.debug(
+              "delete repository directory {}, because of failed repository creation",
+              directory);
+        }
+
+        fileSystem.destroy(directory);
+      }
+
+      Throwables.propagateIfPossible(ex, RepositoryException.class,
+                                     IOException.class);
+    }
   }
 
   /**