Fixed bug of Popen not passing arguments.

carlos-jenkins · carlos-jenkins · commit e93186f2c4b9 · 2014-09-23T18:30:12.000-06:00
Thanks to http://stackoverflow.com/questions/20128114/pythons-subprocess-does-not-interpret-as-expected-on-cygwin/20137299#20137299
diff --git a/webhooks.py b/webhooks.py
@@ -116,7 +116,6 @@ def index():
 
         proc = Popen(
             [s, tmpfile, event],
-            shell=True,
             stdout=PIPE, stderr=PIPE
         )
         stdout, stderr = proc.communicate()

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,6 @@ def index():`
`116`	`116`
`117`	`117`	`proc = Popen(`
`118`	`118`	`[s, tmpfile, event],`
`119`		`- shell=True,`
`120`	`119`	`stdout=PIPE, stderr=PIPE`
`121`	`120`	`)`
`122`	`121`	`stdout, stderr = proc.communicate()`