Skip to content

Follow-up: Improvements for PR #476 (Added Nix development flake)#481

Open
HopHouse wants to merge 2 commits into
Orange-Cyberdefense:mainfrom
HopHouse:feature/nix-dev-flake
Open

Follow-up: Improvements for PR #476 (Added Nix development flake)#481
HopHouse wants to merge 2 commits into
Orange-Cyberdefense:mainfrom
HopHouse:feature/nix-dev-flake

Conversation

@HopHouse
Copy link
Copy Markdown

Important

Please consider merging into t3rp/feature/nix-dev-flake or let me know if maintainers prefer a direct push.

Context

This PR contains improvements for @t3rp's PR #476.

Initially, I encountered some issues in using the correct Python version which is both:

  • Available on my OS
  • Compatible with venv requirements

I did not have Docker/Podman installed on my machine, so I created a Nix flake for my Proxmox environment.
Upon seeing @t3rp's Pull Request, I decided to continue its work and merge it with mine.

Solution

Merge @t3rp flake and mine to:

  • Add Proxmox support
  • Pin the Python version
  • Make it available on all systems through flake-utils.lib.eachDefaultSystem
  • Define devShells and Apps separately
  • Add the flake.lock to make it fully reproducible
  • Keep as much as possible the code from @t3rp

Testing

All the derivation works well on my machine.

Linked PR

@HopHouse HopHouse changed the title Follow-up: Improvements for PR #476 Follow-up: Improvements for PR #476 (Added Nix development flake) Feb 4, 2026
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