diff options
author | Daiderd Jordan <daiderd@gmail.com> | 2020-01-11 15:06:57 +0100 |
---|---|---|
committer | Daiderd Jordan <daiderd@gmail.com> | 2020-01-11 15:45:41 +0100 |
commit | 307bcb9a8e7a16bfc451e055a620b766df9d3f7d (patch) | |
tree | 577361d7811d30943183899c9aed876fe641ae9c /src | |
parent | 6f046fa39ee941c48515cb2e4d1cbb0a962cfed7 (diff) |
libexpr: show expression in assertion errors
Includes the expression of the condition in the assertion message if
the assertion failed, making assertions much easier to debug. eg.
error: assertion (withPython -> (python2Packages != null)) failed at pkgs/tools/security/nmap/default.nix:11:1
Diffstat (limited to 'src')
-rw-r--r-- | src/libexpr/eval.cc | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/libexpr/eval.cc b/src/libexpr/eval.cc index 1c787645d..3a3810f24 100644 --- a/src/libexpr/eval.cc +++ b/src/libexpr/eval.cc @@ -531,9 +531,9 @@ LocalNoInlineNoReturn(void throwTypeError(const char * s, const ExprLambda & fun throw TypeError(format(s) % fun.showNamePos() % s2 % pos); } -LocalNoInlineNoReturn(void throwAssertionError(const char * s, const Pos & pos)) +LocalNoInlineNoReturn(void throwAssertionError(const char * s, const string & s1, const Pos & pos)) { - throw AssertionError(format(s) % pos); + throw AssertionError(format(s) % s1 % pos); } LocalNoInlineNoReturn(void throwUndefinedVarError(const char * s, const string & s1, const Pos & pos)) @@ -1262,8 +1262,11 @@ void ExprIf::eval(EvalState & state, Env & env, Value & v) void ExprAssert::eval(EvalState & state, Env & env, Value & v) { - if (!state.evalBool(env, cond, pos)) - throwAssertionError("assertion failed at %1%", pos); + if (!state.evalBool(env, cond, pos)) { + std::ostringstream out; + cond->show(out); + throwAssertionError("assertion %1% failed at %2%", out.str(), pos); + } body->eval(state, env, v); } |