Skip to content

Conversation

@juhp
Copy link
Contributor

@juhp juhp commented Jan 6, 2026

This is basically #12805 (nix-daemon.sh) for profile.d/nix.sh.

See #5950 (comment)

  • Add a check to ensure the script executes only once per shell.

This makes bash behave the same way as fish,
which checks __ETC_PROFILE_NIX_SOURCED in both nix.fish and nix-daemon.fish.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

Add a check to ensure the script executes only once per shell.

This makes bash behave the same way as fish, which checks `__ETC_PROFILE_NIX_SOURCED` both for nix.fish and nix-daemon.fish
@juhp
Copy link
Contributor Author

juhp commented Jan 22, 2026

Thanks @convoliution - your approach in #15005 may be better perhaps - I haven't tested.
So I think it is better to review that first.

(Currently in the meantime I did apply the patch here to fedora, but I could drop sooner or later if it causes trouble and particularly if there is a better solution.)

@convoliution
Copy link

convoliution commented Jan 23, 2026

@juhp whatever the community prefers! I'm not too up-to-date on the Nix user base's needs, just felt the effects of #13255 enough to sink some time into what I thought might be a passable solution

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