Skip to content

add basic shell.nix#3

Open
redianthus wants to merge 1 commit into
codex-semantics-library:mainfrom
redianthus:nixshell
Open

add basic shell.nix#3
redianthus wants to merge 1 commit into
codex-semantics-library:mainfrom
redianthus:nixshell

Conversation

@redianthus

@redianthus redianthus commented Feb 27, 2026

Copy link
Copy Markdown

Hi @mlemerre,

It allows me to run dune build @all -p codex and to successfully run the tests.

I plan to package both cudd and pacomb in nixpkgs in the future (and probably codex too).

I did not bother with binsec_codex and frama_c_codex for now.

@mlemerre

Copy link
Copy Markdown
Contributor

Hi Leo,

We already have a shell.nix file in devenv/nix/shell.nix (and nix packages for pacomb and stuff). They are meant for our CI so they may not be optimal (and I don't know nix very well), and maybe yours is better suited? WDYT?

@redianthus

redianthus commented Mar 2, 2026

Copy link
Copy Markdown
Author

Yes, I think they could probably be made simpler but I did not wanted to disturb your CI process.

I think it's fine to have both for now. I can try to make them converge towards a unique file in the future if you agree. Having this simple one first makes it easier for me to contribute. (it's better if the shell.nix is at the root of the repo, it's a convention used a lot and it allows me to simply cd codex and have the right environment setup automatically)

EDIT: pacomb and cudd are now part of nixpkgs, I simplified the file a little bit to use the packaged version.

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