New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Print built derivations as json for build #4182
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks good overall, except that it brings back the intentionally removed behavior of printing the built store paths 🤔
(Ftr, and unless I misunderstood the matter, the issue with printing these paths is that they don't come with any guaranty of not being gc-ed if --no-link
is passed or the symlinks are removed, bringing back some scary memories of use-after-free)
@regnat There was still nobody whould could explain to me why |
I don't want to enter this debate. I didn't state any position, just noted that it might be controversial. But fwiw:
There are cases where it wouldn't work, but |
Thanks @regnat , applied those changes |
@regnat I think including |
No that shouldn't. I guess what's happening is that The reason why it works with |
What are cases where you really do need |
@mkenix Mainly for projects which are independent from nixpkgs. Not everything is fitted for nixpkgs, for example NixDroid. |
Add --json option to nix build to allow machine readable output on stdout with all built derivations Fixes NixOS#1930
Thanks! |
Add --json option to nix build to allow machine readable output on
stdout with all built derivations
Fixes #1930
I think closes #2423 and #1647
Wasn't quite sure is there a better place to put a test?