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
bearssl: init at 0.6 & s6-networking bearssl backend #117136
Conversation
d83a839
to
1f1c0a6
Compare
inherit sha256; | ||
}; | ||
|
||
outputs = [ "bin" "lib" "dev" "out" ]; |
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.
Wouldn't lib
be a more appropriate default output?
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.
bin
is always expected to come first, the code that needs lib usually knows to default to it.
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.
Fair enough :)
install -Dm644 build/libbearssl.so $lib/lib/libbearssl.so | ||
install -Dm644 build/libbearssl.a $lib/lib/libbearssl.a | ||
install -Dm644 -t $dev/include inc/*.h | ||
touch $out |
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.
Isn't $out
a directory propagating lib
and bin
?
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.
It didn’t work without the touch …
Result of 2 packages built successfully:
1 suggestion:
Result of 2 packages built successfully:
1 suggestion:
|
18·27 <Profpatsch> skarnet: would you recommend putting bearssl as the default backend for s6-networking? 18·27 <Profpatsch> uh, bearssl isn’t even packaged 18·27 <Profpatsch> yak shave 18·28 <Profpatsch> skarnet: The current backend uses libressl 18·29 <@skarnet> well at least CAFILE works 18·29 <@skarnet> but yes, I would recommend putting bearssl as the default backend
1f1c0a6
to
1918f20
Compare
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)