Skip to content

xdg: add support for local bin path#9067

Merged
rycee merged 1 commit intonix-community:masterfrom
shikanime:pr9067
Apr 11, 2026
Merged

xdg: add support for local bin path#9067
rycee merged 1 commit intonix-community:masterfrom
shikanime:pr9067

Conversation

@shikanime
Copy link
Copy Markdown
Member

@shikanime shikanime commented Apr 8, 2026

Description

Closes #3357.

This adds xdg.localBinInPath, a boolean option that controls whether
Home Manager prepends $HOME/.local/bin to PATH when xdg.enable is
enabled.

The option defaults to false, preserving the old behavior by default
while allowing users to opt in with xdg.localBinInPath = true;.

Checklist

  • Change is backwards compatible.

  • Code formatted with nix fmt or
    nix-shell -p treefmt nixfmt deadnix keep-sorted nixf-diagnose --run treefmt.

  • Code tested through nix run .#tests -- test-all or
    nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.
    • Generate a news entry. See News
    • Basic tests added. See Tests
  • If this PR adds an exciting new feature or contains a breaking change.

    • Generate a news entry. See News

@shikanime shikanime marked this pull request as draft April 8, 2026 11:54
@shikanime shikanime force-pushed the pr9067 branch 2 times, most recently from bb4ce2f to a2212d8 Compare April 8, 2026 12:29
@shikanime shikanime marked this pull request as ready for review April 8, 2026 12:36
@ambroisie
Copy link
Copy Markdown
Contributor

ambroisie commented Apr 9, 2026

Also linked to #6058 I think (as in, I think the path should be configurable).

Comment thread modules/misc/xdg.nix Outdated
@shikanime shikanime force-pushed the pr9067 branch 2 times, most recently from eab94bf to 35f98fd Compare April 9, 2026 16:18
Signed-off-by: William Phetsinorath <william.phetsinorath@shikanime.studio>

Signed-off-by: William Phetsinorath <william.phetsinorath@shikanime.studio>
Change-Id: I045f07415467ca6d409a6f4d52e753866a6a6964
@rycee rycee merged commit f7f6a55 into nix-community:master Apr 11, 2026
7 checks passed
@rycee
Copy link
Copy Markdown
Member

rycee commented Apr 11, 2026

Thanks! Merged to master now 🙂

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.

xdg module should update home.sessionPath

3 participants