[aur-dev] [PATCH 1/3] scripts/cleanup: use native PHP only

[Dan McGee]

No need to shell out to the system here. Also fix the script so it
actually works.

Signed-off-by: Dan McGee <d...@archlinux.org>
---
 scripts/cleanup |   20 +++++++++++---------
 1 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/scripts/cleanup b/scripts/cleanup
index 4fc9ea2..f287350 100755
--- a/scripts/cleanup
+++ b/scripts/cleanup
@@ -16,21 +16,23 @@ if (empty($dir)) {
 }
 
 set_include_path(get_include_path() . PATH_SEPARATOR . "$dir/lib");
-include("config.inc");
-include("aur.inc");
-include("pkgfuncs.inc");
-
-exec('ls ' . INCOMING_DIR, $files);
+include("config.inc.php");
+include("aur.inc.php");
+include("pkgfuncs.inc.php");
 
 $count = 0;
 
+$files = scandir(INCOMING_DIR);
 foreach ($files as $pkgname) {
-       if (!package_exists($pkgname)) {
-               echo 'Removing ' . INCOMING_DIR . "$pkgname\n";
-               system('rm -r ' . INCOMING_DIR . $pkgname);
+       if ($pkgname == '.' || $pkgname == '..') {
+               continue;
+       }
+       $fullpath = INCOMING_DIR . $pkgname;
+       if (!package_exists($pkgname) && is_dir($fullpath)) {
+               echo 'Removing ' . $fullpath . "\n";
+               rm_tree($fullpath);
                $count++;
        }
 }
 
 echo "\nRemoved $count directories.\n";
-
-- 
1.7.6