New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nixos/x11: Respect XCOMPOSECACHE/XDG_DATA_HOME if set in xsession-wrapper #95019
Conversation
Not sure who can give a (final) opinion on this, so I added some of the recent contributors to this file as potential reviewers. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
3fac5f7
to
8a288d3
Compare
8a288d3
to
baad741
Compare
@adisbladis As someone who recently committed to this file as well: do you have an opinion on this PR? |
LGTM, I'd love to see this get merged. I've been trying to make my |
baad741
to
3a2fb22
Compare
Rebased to fix merge conflicts. |
3a2fb22
to
19215f5
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
19215f5
to
962951e
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
As a single data point: I've been using this as a module override in my config since I started working on the PR and it Works For Me. |
This allows users to set e.g. XCOMPOSECACHE before it's used.
962951e
to
580cf02
Compare
Motivation for this change
This is a revival of #42677 of which the fork of nixpkgs has been removed.
The first commit is to allow user settings in
~/.xsession
to actually affect the process.Compared to the old PR I have applied one fix #42677 (comment) , removed the intermediate variable
xdg_data_home
, and unsetcompose_cache
after use.I am not sure if this would adversely affect anything, but in principle the behaviour doesn't change if the variables are not set.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)