diff options
| author | Xavier Claessens <xavier.claessens@collabora.com> | 2020-07-14 11:57:59 -0400 |
|---|---|---|
| committer | Jussi Pakkanen <jpakkane@gmail.com> | 2020-07-14 20:49:47 +0300 |
| commit | 2353d67c25629da049b192d7fddb3e7851edb8df (patch) | |
| tree | 535984640d4a9e954a738891ae4e44576c20ad6f | |
| parent | 4b728293cd111da3d524a23fb02b457f39e5c406 (diff) | |
| download | meson-2353d67c25629da049b192d7fddb3e7851edb8df.tar.gz | |
wrap: Raise MesonException when git command fails
This avoid printing long backtrace by default, the user already has the
output of the git command printed for debugging purpose since we don't
redirect stdout/stderr.
| -rw-r--r-- | mesonbuild/wrap/wrap.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/mesonbuild/wrap/wrap.py b/mesonbuild/wrap/wrap.py index 54daaf35b..aba220e1b 100644 --- a/mesonbuild/wrap/wrap.py +++ b/mesonbuild/wrap/wrap.py @@ -61,7 +61,10 @@ def quiet_git(cmd: T.List[str], workingdir: str) -> T.Tuple[bool, str]: def verbose_git(cmd: T.List[str], workingdir: str, check: bool = False) -> bool: if not GIT: return False - return git(cmd, workingdir, check=check).returncode == 0 + try: + return git(cmd, workingdir, check=check).returncode == 0 + except subprocess.CalledProcessError: + raise WrapException('Git command failed') def whitelist_wrapdb(urlstr: str) -> urllib.parse.ParseResult: """ raises WrapException if not whitelisted subdomain """ |
