-
-
Notifications
You must be signed in to change notification settings - Fork 14.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
net-cpp: init at 3.1.0 #260869
net-cpp: init at 3.1.0 #260869
Conversation
359d31b
to
0c22987
Compare
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.
Result of nixpkgs-review pr 260869
run on x86_64-linux 1
3 packages built:
- net-cpp
- net-cpp.dev
- net-cpp.doc
LGTM, just a nit
}) | ||
]; | ||
|
||
postPatch = lib.optionalString finalAttrs.doCheck '' |
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.
postPatch = lib.optionalString finalAttrs.doCheck '' | |
postPatch = lib.optionalString finalAttrs.finalPackage.doCheck '' |
This covers the case where doCheck = false;
is set using an override. I remembed the cross people discussing it on matrix.
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.
Quick to do treewide however
Description of changes
Working towards #99090.
net-cpp, a C++11 networking library. Needed by Lomiri's location service.
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/
)