Holding isn't a queue either, it's a tmpdir

[dak.git] / daklib / holding.py

diff --git a/daklib/holding.py b/daklib/holding.py
index b637738a5af73d86f1b978137727407d7d9bcf5c..2f36a696313badb4aad725685f1c1362df6c2765 100644 (file)
--- a/daklib/holding.py
+++ b/daklib/holding.py
@@ -45,7 +45,13 @@ class Holding(object):
             self.initialised = True
 
             self.in_holding = {}
-            self.holding_dir = Config()["Dir::Queue::Holding"]
+            self.holding_dir = Config()["Dir::Holding"]
+            # ftptrainees haven't access to holding, use a temp directory instead
+            if not os.access(self.holding_dir, os.W_OK):
+                self.holding_dir = Config()["Dir::TempPath"]
+
+    def chdir_to_holding(self):
+        os.chdir(self.holding_dir)
 
     def copy_to_holding(self, filename):
         base_filename = os.path.basename(filename)