From deddbe5f15479bc8eee1ae3c92317e8b5d7f7c27 Mon Sep 17 00:00:00 2001
From: Sean Harvey <sean@silverstripe.com>
Date: Wed, 13 Feb 2013 09:59:14 +1300
Subject: [PATCH] Better checking for permissions when running Solr_configure

---
 code/solr/Solr.php | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/code/solr/Solr.php b/code/solr/Solr.php
index 33368da..9aaa5f6 100644
--- a/code/solr/Solr.php
+++ b/code/solr/Solr.php
@@ -102,7 +102,11 @@ class Solr_Configure extends BuildTask {
 				foreach (Solr::get_indexes() as $index => $instance) {
 					$sourceDir = $instance->getExtrasPath();
 					$targetDir = "$local/$index/conf";
-					if (!is_dir($targetDir)) mkdir($targetDir, 0770, true);
+					if (!is_dir($targetDir)) {
+						$worked = @mkdir($targetDir, 0770, true);
+						if(!$worked) echo sprintf('Failed creating target directory %s, please check permissions', $targetDir);
+						return;
+					}
 
 					file_put_contents("$targetDir/schema.xml", $instance->generateSchema());