From 1a4a96ccdc3a30d0407c3f3962d05a4047890bc9 Mon Sep 17 00:00:00 2001 From: Ben Segall Date: Thu, 7 Mar 2024 14:06:09 +0000 Subject: [PATCH] Add link to documentation if reproxy fails to start Bug: b/328269200 Change-Id: I647d8ae222ab073324ce7bf1ad4680242526882a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5345028 Commit-Queue: Ben Segall Reviewed-by: Takuto Ikuta --- reclient_helper.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/reclient_helper.py b/reclient_helper.py index d10dc5166..9727e6a97 100644 --- a/reclient_helper.py +++ b/reclient_helper.py @@ -313,6 +313,15 @@ def workspace_is_on_slow_filesystem(): return False +# pylint: disable=line-too-long +def reclient_setup_docs_url(): + if sys.platform == "darwin": + return "https://chromium.googlesource.com/chromium/src/+/main/docs/mac_build_instructions.md#use-reclient" + if sys.platform.startswith("win"): + return "https://chromium.googlesource.com/chromium/src/+/main/docs/windows_build_instructions.md#use-reclient" + return "https://chromium.googlesource.com/chromium/src/+/main/docs/linux/build_instructions.md#use-reclient" + + @contextlib.contextmanager def build_context(argv, tool): # If use_remoteexec is set, but the reclient binaries or configs don't @@ -369,6 +378,11 @@ def build_context(argv, tool): elapsed = time.time() - start print('%1.3f s to start reproxy' % elapsed) if reproxy_ret_code != 0: + print(f'''Failed to start reproxy! +See above error message for details. +Ensure you have completed the reproxy setup instructions: +{reclient_setup_docs_url()}''', + file=sys.stderr) yield reproxy_ret_code return try: