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
compactor: init at 0.11.0 #37680
compactor: init at 0.11.0 #37680
Conversation
@GrahamcOfBorg build compactor |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: compactor Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: compactor Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: compactor Partial log (click to expand)
|
@zimbatm This is the |
As a quick note, I used bundix to generate cbor-diag and cddl. I based them on other tools already in nixpkgs like https://github.com/NixOS/nixpkgs/blob/3ed5af74a5a07e7d38588acbaa02bf921112c068/pkgs/development/tools/cucumber/default.nix |
|
||
bundlerEnv rec { | ||
name = "cddl-${version}"; | ||
version = (import gemset).cddl.version; |
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.
using pname = "cddl"
should be enough
@@ -0,0 +1,18 @@ | |||
{ lib, bundlerEnv, ruby }: | |||
|
|||
bundlerEnv rec { |
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.
this should be a bundlerApp if it just exports a 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.
@fdns is it used just for the command in bin?
@7c6f434c the general rule is that a bundlerEnv is permeable and use to expose libraries to the app. A bundlerApp is sealed and used to expose binaries to the user. |
Yes, it is only for the bin command (It will fail the check without them).
I haven't had time to send and test the change, probably in a day or two.
|
I changed the bundlerEnv to bundlerApp to seal the binaries. I also tested again and all checks are passing. |
Motivation for this change
Adding compactor, a DNS traffic capturer
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)