Skip to content
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

flake.nix: Added nix flake #480

Closed
wants to merge 3 commits into from

Conversation

felix-ulonska
Copy link
Contributor

@felix-ulonska felix-ulonska commented Dec 11, 2024

DO NOT MERGE BEFORE #478

cwe_checker can now be used with Nix flakes.

Now, cwe_checker can be used without cloning:
nix run github:fkie-cad/cwe_checker -- BINARY (after merge)
nix run github:felix-ulonska/cwe_checker/nix-flake -- BINARY

Altenrative in cloned folder
nix run .# -- BINARY

Additionally, now a dev shell can be used:
nix develop in cloned repo. In the opened shell, a compatible rust version is installed, and compatible Ghidra. cwe_chekcer can be start with the following:
cargo run --bin cwe_checker -- BINARY

Most IDEs work when started from within the dev shell.

Valentin Obst and others added 2 commits December 10, 2024 13:00
If config files or Ghidra plugins are not found in the respective project
directories, fall back to env vars that contain an alternative search
directory.
Copy link
Collaborator

@vobst vobst left a comment

Choose a reason for hiding this comment

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

Thanks a lot, that's awesome. Only a few small rewordings here and there.

flake.nix Outdated Show resolved Hide resolved
flake.nix Show resolved Hide resolved
flake.nix Outdated Show resolved Hide resolved
flake.nix Outdated Show resolved Hide resolved
flake.nix Outdated Show resolved Hide resolved
@vobst vobst mentioned this pull request Dec 17, 2024
@vobst
Copy link
Collaborator

vobst commented Dec 17, 2024

Thanks! I squashed, rebased and did some minor style fixes. Will be merged in #481.

@vobst vobst closed this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants