From e09b05e6181f9e7e8dc169407b8bf0d46cf6be2d Mon Sep 17 00:00:00 2001 From: Martin Robinson Date: Thu, 23 Mar 2023 13:01:22 +0100 Subject: [PATCH] Be more verbose when a command fails in the WPT upstream script --- etc/ci/upstream-wpt-changes/wptupstreamer/__init__.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/etc/ci/upstream-wpt-changes/wptupstreamer/__init__.py b/etc/ci/upstream-wpt-changes/wptupstreamer/__init__.py index f1b1ef92598..156bb108c5a 100644 --- a/etc/ci/upstream-wpt-changes/wptupstreamer/__init__.py +++ b/etc/ci/upstream-wpt-changes/wptupstreamer/__init__.py @@ -60,9 +60,14 @@ class LocalGitRepo: env.setdefault("GIT_AUTHOR_NAME", self.sync.github_name) env.setdefault("GIT_COMMITTER_NAME", self.sync.github_name) - return subprocess.check_output( - command_line, cwd=self.path, env=env, stderr=subprocess.STDOUT - ).decode("utf-8") + try: + return subprocess.check_output( + command_line, cwd=self.path, env=env, stderr=subprocess.STDOUT + ).decode("utf-8") + except subprocess.CalledProcessError as exception: + logging.warning("Process execution failed with output:\n%s", + exception.output.decode("utf-8")) + raise exception @dataclasses.dataclass()