Skip to content
This repository was archived by the owner on Aug 1, 2019. It is now read-only.
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 4 additions & 6 deletions git_review/cmd.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,17 +128,18 @@ def run_command_status(*argv, **kwargs):
else:
argv = shlex.split(str(argv[0]))
stdin = kwargs.pop('stdin', None)
print_directly = kwargs.pop('print_directly', False)
newenv = os.environ.copy()
newenv['LANG'] = 'C'
newenv['LANGUAGE'] = 'C'
newenv.update(kwargs)
p = subprocess.Popen(argv,
stdin=subprocess.PIPE if stdin else None,
stdout=subprocess.PIPE,
stdout=None if print_directly else subprocess.PIPE,
stderr=subprocess.STDOUT,
env=newenv, universal_newlines=True)
(out, nothing) = p.communicate(stdin)
return (p.returncode, out.strip())
return (p.returncode, '' if print_directly else out.strip())


def run_command(*argv, **kwargs):
Expand Down Expand Up @@ -239,15 +240,12 @@ def run_custom_script(action):
os.path.join(git_dir, "hooks", script_file)]
for fpath in paths:
if os.path.isfile(fpath) and os.access(fpath, os.X_OK):
status, output = run_command_status(fpath)
status, output = run_command_status(fpath, print_directly=True)
returns.append((status, output, fpath))

for (status, output, path) in returns:
if status:
raise CustomScriptException(status, output, [path], {})
elif output and VERBOSE:
print("script %s output is:" % (path))
print(output)


def git_config_get_value(section, option, default=None, as_bool=False):
Expand Down