Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix default.nix after nixpkgs vulnix path changed #102

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

henrirosten
Copy link
Contributor

@henrirosten henrirosten commented Nov 22, 2024

Vulnix path in nixpkgs was changed with NixOS/nixpkgs#354531. After that change, the vulnix path hardcoded in default.nix no longer matches '{nixpkgs}/pkgs/tools/security/vulnix', which broke the default.nix in vulnix repo.

This PR fixes the issue by referencing nixpkgs vulnix with pkgs.vulnix instead of hardcoding the path.

Signed-off-by: Henri Rosten <henri.rosten@unikie.com>
@henrirosten henrirosten marked this pull request as ready for review November 22, 2024 07:40
Copy link
Member

@zimbatm zimbatm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@zimbatm zimbatm merged commit 8464528 into nix-community:master Nov 22, 2024
2 checks passed
@henrirosten henrirosten deleted the fix-default-nix branch December 20, 2024 12:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants