Skip to content
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

fix: relink the X11 socket into place if needed #207

Merged
merged 3 commits into from
Jan 30, 2023

Conversation

K900
Copy link
Contributor

@K900 K900 commented Jan 29, 2023

Fixes X11 applications on tmp-as-tmpfs setups.

Fixes X11 applications on tmp-as-tmpfs setups.
@SuperSandro2000
Copy link
Member

What is when upstream changes it's tmpfiles.d rule again? Then we have the same problem we had when we removed the rule in #172

@K900
Copy link
Contributor Author

K900 commented Jan 29, 2023

Looks like upstream is actually just masking the default settings now:

~
❯ cat /run/tmpfiles.d/x11.conf
# Note: This file is generated by WSL to prevent systemd-tmpfiles from removing /tmp/.X11-unix during boot.

So we should be OK to recreate the link.

@nzbr
Copy link
Member

nzbr commented Jan 29, 2023

@K900 i've opened a PR (K900#1) against your branch. I mount the windows drives to /drv instead of /mnt, so this didn't work for me. Otherwide it looks good

@K900 K900 merged commit 45194dd into nix-community:main Jan 30, 2023
@nzbr nzbr added the bug Something isn't working label Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants