Fixed django-compressor#25 - Catch IOErrors in the YUI and closure filter.

Author: jezdez

compressor/filters/closure.py

@@ -14,20 +14,22 @@ def output(self, **kwargs):
 
         command = '%s %s' % (BINARY, arguments)
 
-        p = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
-        p.stdin.write(self.content)
-        p.stdin.close()
+        try:
+            p = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
+            p.stdin.write(self.content)
+            p.stdin.close()
 
-        filtered = p.stdout.read()
-        p.stdout.close()
+            filtered = p.stdout.read()
+            p.stdout.close()
 
-        err = p.stderr.read()
-        p.stderr.close()
+            err = p.stderr.read()
+            p.stderr.close()
+        except IOError, e:
+            raise FilterError(e)
 
         if p.wait() != 0:
             if not err:
                 err = 'Unable to apply Closure Compiler filter'
-
             raise FilterError(err)
 
         if self.verbose:

compressor/filters/yui.py

@@ -22,20 +22,23 @@ def output(self, **kwargs):
         if self.verbose:
             command += ' --verbose'
 
-        p = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
-        p.stdin.write(self.content)
-        p.stdin.close()
+        try:
+            p = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
+            p.stdin.write(self.content)
+            p.stdin.close()
 
-        filtered = p.stdout.read()
-        p.stdout.close()
+            filtered = p.stdout.read()
+            p.stdout.close()
 
-        err = p.stderr.read()
-        p.stderr.close()
+            err = p.stderr.read()
+            p.stderr.close()
+
+        except IOError, e:
+            raise FilterError(e)
 
         if p.wait() != 0:
             if not err:
                 err = 'Unable to apply YUI Compressor filter'
-
             raise FilterError(err)
 
         if self.verbose: