[Checkins] SVN: zopyx.smartprintng.client/trunk/ - logging spool directory location

[Andreas Jung]

Log message for revision 122535:
  - logging spool directory location
  - check spool directory for write permissions
  
  

Changed:
  U   zopyx.smartprintng.client/trunk/docs/HISTORY.txt
  U   zopyx.smartprintng.client/trunk/zopyx/smartprintng/client/zip_client.py

-=-
Modified: zopyx.smartprintng.client/trunk/docs/HISTORY.txt
===================================================================
--- zopyx.smartprintng.client/trunk/docs/HISTORY.txt	2011-08-11 08:37:31 UTC (rev 122534)
+++ zopyx.smartprintng.client/trunk/docs/HISTORY.txt	2011-08-11 09:58:17 UTC (rev 122535)
@@ -1,6 +1,11 @@
 Changelog
 =========
 
+0.8.1 (2011-08-11)
+------------------
+- logging spool directory location
+- check spool directory for write permissions
+
 0.8.0 (2011-08-02)
 ------------------
 - obsoleted Proxy() implementation

Modified: zopyx.smartprintng.client/trunk/zopyx/smartprintng/client/zip_client.py
===================================================================
--- zopyx.smartprintng.client/trunk/zopyx/smartprintng/client/zip_client.py	2011-08-11 08:37:31 UTC (rev 122534)
+++ zopyx.smartprintng.client/trunk/zopyx/smartprintng/client/zip_client.py	2011-08-11 09:58:17 UTC (rev 122535)
@@ -14,8 +14,11 @@
 import tempfile
 import zipfile
 import warnings
+import logging
 
+LOG = logging.getLogger()
 
+
 class Proxy(object):
     """ THIS IMPLEMENTATION IS OBSOLETE, USE 'Proxy2' INSTEAD """
 
@@ -30,6 +33,14 @@
         if not os.path.exists(output_directory):
             os.makedirs(output_directory)
         self.output_directory = output_directory
+        # check if directory is writeable
+        try:
+            tmpname = os.path.join(self.output_directory, 'test')
+            file(tmpname, 'w').write('foo')
+            os.unlink(tmpname)
+        except IOError:
+            raise IOError('Spool directory %s is not writeable' % self.output_directory)
+        LOG.info('Using spool directory %s' % self.output_directory)
 
     def _makeZipFromDirectory(self, directory):
         """ Generate a ZIP file from a directory containing all its
@@ -136,6 +147,14 @@
         if not os.path.exists(output_directory):
             os.makedirs(output_directory)
         self.output_directory = output_directory
+        # check if directory is writeable
+        try:
+            tmpname = os.path.join(self.output_directory, 'test')
+            file(tmpname, 'w').write('foo')
+            os.unlink(tmpname)
+        except IOError:
+            raise IOError('Spool directory %s is not writeable' % self.output_directory)
+        LOG.info('Using spool directory %s' % self.output_directory)
 
     def _makeZipFromDirectory(self, directory):
         """ Generate a ZIP file from a directory containing all its