Replies: 2 comments
-
Sounds good, thanks for the offer. Can you bring PRs ready to merge to this Discussion for now? |
Beta Was this translation helpful? Give feedback.
-
Sure, I'll go through and pick some out. Looks like some repo tests were broken, but were fixed in nix-community/nixGL#201. Looks like there's no way for me to re-run them outside of creating a new PR. But they don't do much anyway - they just build, they don't confirm that it actually works. I filed nix-community/nixGL#205 about automating the current integration test. https://github.com/nix-community/nixGL/pull/195/files would be mergeable with the cachix change removed. After that change, we will test against a newer nvidia driver, though still not the newest. nix-community/nixGL#131 could be merged as-is but would benefit from being run again to update the drivers JSON. There's other ways to implement it that might be less fragile (it parses HTML to get the versions, but they also have an XML API), but if it's working I wouldn't be inclined to require a rewrite to that before merging. nix-community/nixGL#190 is ready to merge. nix-community/nixGL#187 is ready to merge. There's a few more PRs I haven't looked at yet but I'm going to leave them for later. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
nixGL has been unmaintained for a while since the original developer handed it off to nix-community. There's a lot of issues and PRs backed up now and, in particular, the new Mesa version 24.3 added a library which is not correctly handled by nixGL - see nix-community/nixGL#190 for more information. This is going to bite more and more people as people upgrade from older versions of nixpkgs.
Another issue is that there's currently no license for the project. That should be fixed before resuming development. See nix-community/nixGL#143 for more about that. There is an open PR, nix-community/nixGL#174, which would add a license file, but it still needs sign-off from all the significant contributors.
I'm not sure whether the author @jeff-hykin is up for bringing the license issue to a close as it sat for a while without discussion. If not, I'm willing to pick that up. I don't think I need permissions on the repo for that as it's 99% talking to people and 1% clicking the merge button.
Beta Was this translation helpful? Give feedback.
All reactions