diff options
| author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2019-01-16 15:09:41 -0500 |
|---|---|---|
| committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2019-01-16 19:28:03 -0500 |
| commit | d212cdfa6e07abb83e527dcd99d5f6638be942ea (patch) | |
| tree | eef1ac4b2b3af5d2530e317c11aa4074b229e914 /buildstream/_exceptions.py | |
| parent | d34a4fd17028e87d5b9402e59e84b803aa475635 (diff) | |
| download | buildstream-tristan/error-message-regression.tar.gz | |
sandbox/sandbox.py: Display failed commands in the detail stringtristan/error-message-regression
We should only display commands in detail strings, not in the
message texts.
This also updates tests/integration/sandbox-bwrap.py to expect
the new message string which only contains the command exit status
and not the whole command itself, this does not alter the validity
of the text case which is checking that we can obtain the expected
return value.
Diffstat (limited to 'buildstream/_exceptions.py')
| -rw-r--r-- | buildstream/_exceptions.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/buildstream/_exceptions.py b/buildstream/_exceptions.py index ea5ea62f2..6d8ea6d38 100644 --- a/buildstream/_exceptions.py +++ b/buildstream/_exceptions.py @@ -262,8 +262,8 @@ class PlatformError(BstError): # Raised when errors are encountered by the sandbox implementation # class SandboxError(BstError): - def __init__(self, message, reason=None): - super().__init__(message, domain=ErrorDomain.SANDBOX, reason=reason) + def __init__(self, message, detail=None, reason=None): + super().__init__(message, detail=detail, domain=ErrorDomain.SANDBOX, reason=reason) # ArtifactError |