fromupstream: Don't change 'commit' type
The python linter complains that this script changes 'commit' from a
string type to a tuple. Let's just drop the usage of the variable in
these process() calls because we don't use the result.
BUG=None
TEST=Run linter and see that this warning is gone
Change-Id: I84c1e45943fdf6db2a7a822372eafdfa5b15ffc4
Signed-off-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1329762
Reviewed-by: Brian Norris <briannorris@chromium.org>
diff --git a/contrib/fromupstream.py b/contrib/fromupstream.py
index bc83943..b2b69ea 100755
--- a/contrib/fromupstream.py
+++ b/contrib/fromupstream.py
@@ -311,7 +311,7 @@
extra = ['-s']
else:
extra = []
- commit = subprocess.Popen(
+ subprocess.Popen(
['git', 'commit'] + extra + ['--amend', '-F', '-'],
stdin=subprocess.PIPE
).communicate(commit_message)
@@ -331,7 +331,7 @@
commit_message = args['tag'] + commit_message
# commit everything
- commit = subprocess.Popen(
+ subprocess.Popen(
['git', 'commit', '--amend', '-F', '-'], stdin=subprocess.PIPE
).communicate(commit_message)