search

nix-community / comma

Comma runs software without installing it. [maintainers=@Artturin,@burke,@DavHau]
MIT License
1.14k stars 50 forks source link

Print warnings to stderr #58

Closed athre0z closed 1 year ago

athre0z commented 1 year ago

With the prevoius behavior any warning being printed would break things when piping command output to files.

For example I tried running:

, dot -Tsvg < mir_dump/somefunc.nll.0.dot > blah.svg

with an outdated nix-index DB and ended up with an SVG starting with

Warning: Nix-index database is older than 30 days

I think it's better to print these to stderr. There is one println to stdout left for the --print-package argument that I didn't change because I wasn't sure whether that belongs to stderr or stdout, so I left it for now.

Artturin commented 1 year ago

Thanks, --print-package should go to stdout.