From e507e4f9d95aa34076e826418d216bfccd2f0a06 Mon Sep 17 00:00:00 2001
From: Jan Steemann <jan@arangodb.com>
Date: Mon, 13 Apr 2015 11:45:16 +0200
Subject: [PATCH] issue #1300: do not make arangodump fail if output directory
 exists but is empty

---
 arangosh/V8Client/arangodump.cpp | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/arangosh/V8Client/arangodump.cpp b/arangosh/V8Client/arangodump.cpp
index 8e3721d50f..cf9e494366 100644
--- a/arangosh/V8Client/arangodump.cpp
+++ b/arangosh/V8Client/arangodump.cpp
@@ -1287,8 +1287,17 @@ int main (int argc, char* argv[]) {
   // .............................................................................
 
   bool isDirectory = false;
+  bool isEmptyDirectory = false;
+
   if (! OutputDirectory.empty()) {
     isDirectory = TRI_IsDirectory(OutputDirectory.c_str());
+    
+    if (isDirectory) {  
+      TRI_vector_string_t files = TRI_FullTreeDirectory(OutputDirectory.c_str());
+      // we don't care if the target directory is empty
+      isEmptyDirectory = (files._length == 0);
+      TRI_DestroyVectorString(&files); 
+    }
   }
 
   if (OutputDirectory.empty() ||
@@ -1297,7 +1306,7 @@ int main (int argc, char* argv[]) {
     TRI_EXIT_FUNCTION(EXIT_FAILURE, nullptr);
   }
 
-  if (isDirectory && ! Overwrite) {
+  if (isDirectory && ! isEmptyDirectory && ! Overwrite) {
     cerr << "output directory '" << OutputDirectory << "' already exists. use \"--overwrite true\" to overwrite data in it" << endl;
     TRI_EXIT_FUNCTION(EXIT_FAILURE, nullptr);
   }
@@ -1453,9 +1462,7 @@ int main (int argc, char* argv[]) {
     ret = EXIT_FAILURE;
   }
 
-  if (Client != nullptr) {
-    delete Client;
-  }
+  delete Client;
 
   TRIAGENS_REST_SHUTDOWN;