Chromium Code Reviews| Index: tools/clang/scripts/upload_revision.py |
| diff --git a/tools/clang/scripts/upload_revision.py b/tools/clang/scripts/upload_revision.py |
| index b6331df6cdb55a00a4b027da7179de3ba59498fb..8481616c08dc4333f46ebe761348e75f131759fb 100755 |
| --- a/tools/clang/scripts/upload_revision.py |
| +++ b/tools/clang/scripts/upload_revision.py |
| @@ -39,7 +39,9 @@ def PatchRevision(clang_revision, clang_sub_revision): |
| def Git(args): |
| - subprocess.check_call(["git"] + args) |
| + is_win = sys.platform.startswith('win32') |
|
scottmg
2016/09/07 22:00:45
Maybe make is_win (or IS_WIN) a global.
Nico
2016/09/07 22:01:53
Done.
|
| + # Needs shell=True on Windows due to gn.bat in depot_tools. |
|
scottmg
2016/09/07 22:00:45
...due to git.bat...
Nico
2016/09/07 22:01:53
Done.
|
| + subprocess.check_call(["git"] + args, shell=is_win) |
| def main(): |
| parser = argparse.ArgumentParser(description='upload new clang revision') |
| @@ -54,8 +56,10 @@ def main(): |
| clang_revision = args.clang_revision[0] |
| clang_sub_revision = args.clang_sub_revision |
| + is_win = sys.platform.startswith('win32') |
| + # Needs shell=True on Windows due to gn.bat in depot_tools. |
| git_revision = subprocess.check_output( |
| - ["git", "rev-parse", "origin/master"]).strip() |
| + ["git", "rev-parse", "origin/master"], shell=is_win).strip() |
| print "Making a patch for Clang revision r{}-{}".format( |
| clang_revision, clang_sub_revision) |
| print "Chrome revision: {}".format(git_revision) |