Skip to content

Conversation

@nim65s
Copy link

@nim65s nim65s commented Jun 2, 2025

Hi,

This add a simplified direnv module.

It is good enough for my use cases, but I have no idea if this is in the scope of this project or if I should do this elsewhere.

fix #199

@nim65s nim65s marked this pull request as draft June 2, 2025 17:05
@nim65s nim65s marked this pull request as ready for review June 2, 2025 17:10
@nim65s nim65s changed the title add simple direnv support, fix #199 add simple direnv support Jun 3, 2025
@zimbatm
Copy link
Member

zimbatm commented Jun 14, 2025

Nice, thank you.

One idea we had was that system-manager should expose enough NixOS-like stubs that it becomes possible to import modules from NixOS directly. The coverage isn't complete right now, but maybe imports = [ "${nixpkgs}/nixos/modules/programs/direnv.nix" ] works?

Alternatively, we could also merge system-manager modules, but then we should probably use some sort of prefix so they don't clash with NixOS modules. Something short like sm..

@nim65s
Copy link
Author

nim65s commented Jun 14, 2025

Great, thanks for the answer !

I can't test that for the next 2 weeks, so if anybody want to give it a try please don't hesitate.

Otherwise, I'll get back to this in July :)

@r-vdp
Copy link
Member

r-vdp commented Aug 18, 2025

Did this work out with the upstream module in the end, or do we still want to merge this PR?

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.

programs.direnv integration

3 participants