Improve gerrit recipe_module example

The /a suffix in the hots URL is not necessary as the gerrit_util adds it by
default to authenticate. Having it hard-coded in the example can be misleading.

Bug: 685318
Change-Id: I333cd8b2aa9020aadfd186f2e18fbff0aa917681
Reviewed-on: https://chromium-review.googlesource.com/564611
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
changes/11/564611/2
Michael Achenbach 8 years ago committed by Commit Bot
parent 79e897b017
commit 9cb802ca48

@ -6,7 +6,7 @@
"RECIPE_PACKAGE_REPO[depot_tools]/gerrit_client.py",
"branch",
"--host",
"https://chromium-review.googlesource.com/a",
"https://chromium-review.googlesource.com",
"--project",
"v8/v8",
"--branch",
@ -37,7 +37,7 @@
"RECIPE_PACKAGE_REPO[depot_tools]/gerrit_client.py",
"branchinfo",
"--host",
"https://chromium-review.googlesource.com/a",
"https://chromium-review.googlesource.com",
"--project",
"v8/v8",
"--branch",
@ -65,7 +65,7 @@
"RECIPE_PACKAGE_REPO[depot_tools]/gerrit_client.py",
"changes",
"--host",
"https://chromium-review.googlesource.com/a",
"https://chromium-review.googlesource.com",
"--json_file",
"/path/to/tmp/json",
"--start",

@ -8,7 +8,7 @@ DEPS = [
def RunSteps(api):
host = 'https://chromium-review.googlesource.com/a'
host = 'https://chromium-review.googlesource.com'
project = 'v8/v8'
branch = 'test'

Loading…
Cancel
Save