Merge pull request jazzband#590 from solkaz/master

Remove empty args from argument_list in SubprocessCompiler

Author: cyberdelia

AUTHORS

@@ -59,6 +59,7 @@ or just made Pipeline more awesome.
  * Jannis Leidel <[email protected]>
  * Jared Scott <[email protected]>
  * Jaromir Fojtu <[email protected]>
+ * Jeff Held <[email protected]>
  * Jon Dufresne <[email protected]>
  * Josh Braegger <[email protected]>
  * Joshua Kehn <[email protected]>

pipeline/compilers/__init__.py

@@ -112,6 +112,9 @@ def execute_command(self, command, cwd=None, stdout_captured=None):
             else:
                 argument_list.extend(flattening_arg)
 
+        # The first element in argument_list is the program that will be executed; if it is '', then
+        # a PermissionError will be raised. Thus empty arguments are filtered out from argument_list
+        argument_list = filter(None, argument_list)
         stdout = None
         try:
             # We always catch stdout in a file, but we may not have a use for it.

tests/tests/test_compiler.py

@@ -42,6 +42,18 @@ def compile_file(self, infile, outfile, outdated=False, force=False):
         )
         return self.execute_command(command)
 
+class CompilerWithEmptyFirstArg(SubProcessCompiler):
+    output_extension = 'junk'
+
+    def match_file(self, path):
+        return path.endswith('.coffee')
+
+    def compile_file(self, infile, outfile, outdated=False, force=False):
+        command = (
+            ('', '/usr/bin/env', 'cat'),
+            infile,
+        )
+        return self.execute_command(command, stdout_captured=outfile)
 
 class CopyingCompiler(SubProcessCompiler):
     output_extension = 'junk'
@@ -149,6 +161,20 @@ def tearDown(self):
         default_collector.clear()
 
 
+@pipeline_settings(COMPILERS=['tests.tests.test_compiler.CompilerWithEmptyFirstArg'])
+class CompilerWithEmptyFirstArgTest(TestCase):
+    def setUp(self):
+        default_collector.collect()
+        self.compiler = Compiler()
+
+    def test_compile(self):
+            paths = self.compiler.compile([_('pipeline/js/dummy.coffee')])
+            default_collector.collect()
+            self.assertEqual([_('pipeline/js/dummy.junk')], list(paths))
+
+    def tearDown(self):
+        default_collector.clear()
+
 @pipeline_settings(COMPILERS=['tests.tests.test_compiler.InvalidCompiler'])
 class InvalidCompilerTest(TestCase):
     def setUp(self):
@@ -158,7 +184,6 @@ def setUp(self):
     def test_compile(self):
         with self.assertRaises(CompilerError) as cm:
             self.compiler.compile([_('pipeline/js/dummy.coffee')])
-
             e = cm.exception
             self.assertEqual(
                 e.command,