From f440558acc76c2939cf5f0744c6669279b6351a0 Mon Sep 17 00:00:00 2001 From: Eelco Dolstra Date: Thu, 17 Oct 2013 11:47:38 +0200 Subject: [PATCH] Don't show in stack traces Messages like while evaluating the attribute `outPath' at `/nix/store/212ngf4ph63mp6p1np2bapkfikpakfv7-nix-1.6/share/nix/corepkgs/derivation.nix:18:9': are redundant, because Nix already shows that it's evaluating a derivation: while instantiating the derivation named `firefox-24.0' at `/home/eelco/Dev/nixpkgs/pkgs/applications/networking/browsers/firefox/default.nix:131:5': while evaluating the derivation attribute `nativeBuildInputs' at `/home/eelco/Dev/nixpkgs/pkgs/stdenv/generic/default.nix:76:17': --- src/libexpr/eval.cc | 2 +- src/libexpr/eval.hh | 1 + src/libexpr/primops.cc | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/libexpr/eval.cc b/src/libexpr/eval.cc index a3aeeb55c..59c42d0b0 100644 --- a/src/libexpr/eval.cc +++ b/src/libexpr/eval.cc @@ -639,7 +639,7 @@ void ExprSelect::eval(EvalState & state, Env & env, Value & v) state.forceValue(*vAttrs); } catch (Error & e) { - if (pos) + if (pos && pos->file != state.sDerivationNix) addErrorPrefix(e, "while evaluating the attribute `%1%' at %2%:\n", showAttrPath(attrPath), *pos); throw; diff --git a/src/libexpr/eval.hh b/src/libexpr/eval.hh index 2f87b9c77..8707182ed 100644 --- a/src/libexpr/eval.hh +++ b/src/libexpr/eval.hh @@ -95,6 +95,7 @@ public: const Symbol sWith, sOutPath, sDrvPath, sType, sMeta, sName, sSystem, sOverrides, sOutputs, sOutputName, sIgnoreNulls; + Symbol sDerivationNix; /* If set, force copying files to the Nix store even if they already exist there. */ diff --git a/src/libexpr/primops.cc b/src/libexpr/primops.cc index a216fa7ab..0585f5bbe 100644 --- a/src/libexpr/primops.cc +++ b/src/libexpr/primops.cc @@ -1262,7 +1262,9 @@ void EvalState::createBaseEnv() /* Add a wrapper around the derivation primop that computes the `drvPath' and `outPath' attributes lazily. */ - evalFile(findFile("nix/derivation.nix"), v); + string path = findFile("nix/derivation.nix"); + sDerivationNix = symbols.create(path); + evalFile(path, v); addConstant("derivation", v); /* Now that we've added all primops, sort the `builtins' attribute