-
-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
irrd: 4.4.2 -> 4.4.4; fix build #319522
irrd: 4.4.2 -> 4.4.4; fix build #319522
Conversation
Suggestions:
Self-promotion:
|
Result of 2 packages failed to build:
Failure could be specific to my host, idk yet. This is second build that fails. |
No, some tests are still failing. I'm working on it. |
@superherointj where is this order you are referring to defined? |
By defined, if you meant mandatory, I don't know, likely isn't. But it is usual to list entries by phases order which is the order of execution. The phases are hooks that happens in sequence. Errors also shows up in sequence too. In case, you don't want to do this, no problem. |
7b218f3
to
01b8c92
Compare
I think the phases/definitions should be in order of execution now. If not, tell me what I need to change. Also the build is fixed now. |
I disagree, I prefer the
I disagree. The commit does two things at once and both should be visible from reading the first line. |
Result of 2 packages built:
|
Not a blocker. But to clarify, nested |
Successfully created backport PR for |
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.