gclient: Stop printing newline in front of "________ running hook"
The newline is probably here to protect against a hook that doesn't print
a trailing newline. I've never seen a hook that did that, and if we found one,
we could make the logic look like "print a trailing newline if it's not there"
-- or just fix the hook.
The newline has been around since depot_tools was created
(https://codereview.chromium.org/92087, gclient.py), back then this was in
a general "run stuff" function, not in hooks-specific code. Maybe it made
more sense back then.
This is part of a few changes to make `gclient runhooks` less noisy.
Bug: 772741
Change-Id: I285f76dc3f01c5acf5bbaa0be4db9f6edb9c0366
Reviewed-on: https://chromium-review.googlesource.com/706914
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
diff --git a/gclient_utils.py b/gclient_utils.py
index ac7494c..90f577b 100644
--- a/gclient_utils.py
+++ b/gclient_utils.py
@@ -291,7 +291,7 @@
"""
stdout = kwargs.setdefault('stdout', sys.stdout)
if header is None:
- header = "\n________ running '%s' in '%s'\n" % (
+ header = "________ running '%s' in '%s'\n" % (
' '.join(args), kwargs.get('cwd', '.'))
if always: