nix icon indicating copy to clipboard operation
nix copied to clipboard

Changes to make Nix output more informative errors

Open Mathnerd314 opened this issue 8 years ago • 17 comments

  • Output several more locations during --show-trace
  • Option -C / --color for use with less -r
  • nix-instantiate respects '-k' by continuing evaluation after errors (only with derivations)

Mathnerd314 avatar Aug 23 '15 05:08 Mathnerd314

@edolstra Is there any possibility of this being merged?

Mathnerd314 avatar Sep 04 '15 17:09 Mathnerd314

Don't leave us hanging! You want to make nix output more what? :smile:

copumpkin avatar Sep 10 '15 23:09 copumpkin

@edolstra didn't read the code itself, but this is worth persuing

domenkozar avatar Oct 08 '15 10:10 domenkozar

:+1:

copumpkin avatar Nov 18 '15 11:11 copumpkin

Another error that needs reasoning explanation:

error: access to path ‘/home/domen/nixpkgs/nixos/release-combined.nix’ is forbidden in restricted mode

It should at least say that it will evaluate nix only in search path and in Nix store.

domenkozar avatar Dec 28 '15 08:12 domenkozar

This would be so nice to have. @Mathnerd314 could you resolve merge errors and I'll test?

domenkozar avatar Feb 23 '16 14:02 domenkozar

I might update this eventually, but not anytime soon. There are some systematic design flaws which I want to resolve first.

Mathnerd314 avatar Feb 27 '16 22:02 Mathnerd314

Any update on this?

alesya-h avatar Apr 18 '16 01:04 alesya-h

@alesguzik I've stopped most Nix development until June 24, 2016; you're welcome to do something with it before then though.

Mathnerd314 avatar Apr 18 '16 03:04 Mathnerd314

@Mathnerd314 would love to see this going, best to open one PR per fix. That way there is a higher chance they will get merged.

domenkozar avatar Jul 21 '16 12:07 domenkozar

@Mathnerd314 could you elaborate on

There are some systematic design flaws which I want to resolve first.

so someone else can pick this up effectively?

copumpkin avatar Oct 06 '17 01:10 copumpkin

@copumpkin That's just #296 and related issues; the only issue with this PR I know of is that I'm too lazy to update it to fix the merge conflicts. From Haskell it's clear that lazy evaluation works, and so far there's been nothing urgent enough in Nix development to force me to update. But of course anyone else is welcome to pick up the code and run with it.

Mathnerd314 avatar Oct 08 '17 13:10 Mathnerd314

This would be great! Has anyone looked at tackling the conflicts yet? Hopefully not too bad, but even if they are... breaking this into incremental improvements (as suggested above, and I agree should be done) means progress can be made without fixing all the things.

Is this likely to have costs to eval time/size? I've always assumed --show-trace existed (and was off by default) so that these costs were opt-in for debugging, but never actually checked how/if that's done. Also I have "show-trace = 1" or whatever in my nix.conf haha. Anyway, let's revive this!

dtzWill avatar May 22 '18 12:05 dtzWill

Is there any life left in this?

deliciouslytyped avatar Apr 25 '19 16:04 deliciouslytyped

I marked this as stale due to inactivity. → More info

stale[bot] avatar Feb 13 '21 07:02 stale[bot]

I am sad this was neglected. I started looking to see if there was anything to salvage after our recent log overhauling.

Ericson2314 avatar Mar 01 '22 03:03 Ericson2314

https://github.com/NixOS/nix/compare/master...obsidiansystems:noisy-nix this is the first commit (without the color parts, as we have much more of that now) building but untested. That's enough for now.

Ericson2314 avatar Mar 01 '22 04:03 Ericson2314