-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
stdenv: print the time the phase took if it was longer than 30s #166827
Conversation
Is it intended that there's newlines?
Seems to work though, so that's already looking good. Implementation seems quite straightforward to me. |
will be useful for finding why a build runs for a long time on hydra because of tests or the build etc etc
fixed |
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.
LGTM. using date +%s
opens up possible errors in time reporting if the system clock changes, but that's not something we can easily work around (nor would it matter too much in practice)
@ofborg build stdenv.__bootPackages.perl |
Nice! I think this indeed may be useful sometimes, I'm sure a lot of people will find this helpful. |
will be useful for finding why a build runs for a long time on hydra
because of tests or the build etc etc
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes