From feabbdb44064888a6f0f1bf8bd0049f7162206c9 Mon Sep 17 00:00:00 2001 From: "Shawn O. Pearce" Date: Thu, 19 Mar 2009 10:20:27 -0700 Subject: [PATCH] Don't bother listing branch URLs during upload Modern Gerrit2 automatically outputs the URL for each commit to stderr as it creates the records. Dumping the URL ourselves is unnecessary additional output, and worse is just an approximate guess for the correct web URL. Gerrit might not live at the top level directory for the server, or might even prefer a different hostname for web connections than what is listed in the manifest. Signed-off-by: Shawn O. Pearce --- subcmds/upload.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/subcmds/upload.py b/subcmds/upload.py index 58797507..3e664056 100644 --- a/subcmds/upload.py +++ b/subcmds/upload.py @@ -234,9 +234,6 @@ files and description associated with the change in Gerrit. print >>sys.stderr, '[OK ] %-15s %s' % ( branch.project.relpath + '/', branch.name) - print >>sys.stderr, '%s' % branch.tip_url - print >>sys.stderr, '(as %s)' % branch.owner_email - print >>sys.stderr, '' if have_errors: sys.exit(1)