opendev/git-review
Code Issues Proposed changes

Merge "Remove empty lines from git log output"

Browse Source
This commit is contained in:
Jenkins 2013-11-25 22:19:15 +00:00 committed by Gerrit Code Review
commit e0340fa341
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
git_review/cmd.py
View File

@ -528,7 +528,8 @@ def assert_one_change(remote, branch, yes, have_hook):
print("Had trouble running %s" % cmd)
print(output)
sys.exit(1)
output_lines = len(output.split("\n"))
filtered = filter(None, output.split("\n"))
output_lines = sum(1 for s in filtered)
if output_lines == 1 and not have_hook:
printwrap("Your change was committed before the commit hook was "
"installed. Amending the commit to add a gerrit change id.")