-
-
Notifications
You must be signed in to change notification settings - Fork 128
fix: set GBM_BACKENDS_PATH
#190
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
Conversation
|
Test failure looks spurious. |
|
Works for me, also fixes hyprland |
|
This works for me when I override the nixGL nixpkgs with a more recent version otherwise building fails with Seems that the offending gbm files were introduced in a recent version of mesa. NixOS/nixpkgs#368379 |
This fix was taken from this PR: nix-community#190. This fixes the GBM backend related issues when trying to start Hyprland and some other openGL based apps.
|
Is there a path forward to merge this? It solved my problems trying to run hyprland on a non-NixOS Ubuntu host? |
|
Thanks for the patch, fixed the issue on my machine. Similar to noar-t at Ubuntu 24.04.2 LTS with Intel Arc graphics. |
|
Thank you for this patch. It fixed my issue with Chromium-based bwosers crashing on non-NixOS/generic linux installs in NixOS/nixpkgs#413500 Update: |
|
this seems to also be needed for niri to work on non-NixOS. one of my users reports that this fixes their issue. YaLTeR/niri#1780 (comment) is there any reason it hasn't been merged? seems like a very simple change that fixes an issue affecting, as far as I can tell, probably all compositors? (cage, hyprland, niri) |
+1, can we have someone elaborate on exactly what the blocker is here? If none, can we have this merged? |
|
It looks like nixGL may currently not have any maintainers - the last commit to it was made in April 2024. The previous maintainer transferred ownership to nix-community due to no longer having motivation to work on it. There is also a licensing issue, see #143 #174 https://github.com/orgs/nix-community/discussions/882 for more information about this. I think if someone wanted to get this library working again, they could take up the task of getting the license fixed. Contacting all the prior contributors might also help find people who are interested in helping maintain it. I just started a discussion over on the main nix-community discussion page about getting a maintainer for this package. https://github.com/orgs/nix-community/discussions/1884 |
dyfrgi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests should now pass with #201 merged, but we should re-run.
We don't have good tests to verify this, but several users manually tested this, so I think this is good to merge.
|
Tested manually on Debian 12 with Intel and it's working as-is there. @zimbatm can you please merge this one? |
|
can somebody take a look at failing CI on master? |
This snippet references a file Lines 132 to 138 in a8e1ce7
There's no dr-xr-xr-x - /nix/store/2g3qs3ww7vj5zprl7a8908jik9rq95b4-nvidia-x11-440.82-nixGL/share/vulkan/icd.d
.r--r--r-- 212 └── nvidia_icd.jsonIt also references an There's also a comment acknowledging it probably doesn't work on i686: Line 115 in a8e1ce7
Perhaps a better way to get around this issue would be to change the offending code to glob all files in |
|
This PR seems to break my home-manager settings on Debian 12 with the following output: Reverting to previous commit fixes it. |
|
@ramblehead Could you provide a minimal working example flake that reproduces that error? It might be necessary to conditionally attempt construction of the If you are on |
|
@bb010g this error occurred when updating This is my I just updated my NixOS PCs to |
|
Switching to Perhaps, it worth mentioning somewhere in |

This is necessary to fix
nixGLIntelfor at leastcageon SteamOS with nixos-unstable using the overlay.