Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(59)

Issue 545333002: Make git cl format work on my Windows box. (Closed)

Created:
6 years, 3 months ago by Nico
Modified:
6 years, 3 months ago
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Project:
tools
Visibility:
Public.

Description

Make git cl format work on my Windows box. For me, PATH was a unicode instead of a str for some reason, so things failed with D:\src\chromefetch\src>git cl format Traceback (most recent call last): File "d:\src\depot_tools\git_cl.py", line 2821, in <module> sys.exit(main(sys.argv[1:])) File "d:\src\depot_tools\git_cl.py", line 2807, in main return dispatcher.execute(OptionParser(), argv) File "d:\src\depot_tools\subcommand.py", line 245, in execute return command(parser, args[1:]) File "d:\src\depot_tools\git_cl.py", line 2768, in CMDformat stdout = RunCommand(cmd, stdin=diff_output, cwd=top_dir, env=env) File "d:\src\depot_tools\git_cl.py", line 81, in RunCommand return subprocess2.check_output(args, shell=False, **kwargs) File "d:\src\depot_tools\subprocess2.py", line 515, in check_output return check_call_out(args, stdout=PIPE, **kwargs)[0] File "d:\src\depot_tools\subprocess2.py", line 475, in check_call_out out, returncode = communicate(args, **kwargs) File "d:\src\depot_tools\subprocess2.py", line 449, in communicate proc = Popen(args, **kwargs) File "d:\src\depot_tools\subprocess2.py", line 237, in __init__ super(Popen, self).__init__(args, **kwargs) File "d:\src\depot_tools\python276_bin\lib\subprocess.py", line 709, in __init__ errread, errwrite) File "d:\src\depot_tools\python276_bin\lib\subprocess.py", line 957, in _execute_child startupinfo) TypeError: environment can only contain strings Sending crash report ... args: ['d:\\src\\depot_tools\\git_cl.py', 'format'] cwd: D:\src\chromefetch\src exception: environment can only contain strings host: THAKIS1-W.ad.corp.google.com stack: File "d:\src\depot_tools\git_cl.py", line 2821, user: thakis version: 2.7.6 (default, Nov 10 2013, 19:24:18) This fixes the glitch. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=291825

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M git_cl.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Nico
6 years, 3 months ago (2014-09-05 18:48:13 UTC) #2
iannucci
UNICOOOODEEEE!!! (in python!!!!!!) lgtm
6 years, 3 months ago (2014-09-05 18:50:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/545333002/1
6 years, 3 months ago (2014-09-05 18:52:46 UTC) #5
commit-bot: I haz the power
6 years, 3 months ago (2014-09-05 18:54:23 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 291825

Powered by Google App Engine
This is Rietveld 408576698