Updates logic due to removal of the "objects_list" dictionary in the original patch script #4

Merged

niscolas merged 2 commits from main into main

2024-01-30 14:54:28 +00:00

niscolas commented

2023-12-19 00:51:42 +00:00

(Migrated from github.com)

I've tested it and it works on my system, would be great to have more people to test it.
The original script recently removed the "objects_list" dictionary and replaced with a quick "if" based on the version of the Nvidia Driver. The result can be either "libnvcuvid.so" or "libnvidia-encode.so".
I'm really new to Nix so I wasn't able to come up with a solution to determine the "object" automatically based on the driver version (this is how it's done on the original script), so help on that would be awesome! (And pretty quick and easy for any experienced Nix user)

I've tested it and it works on my system, would be great to have more people to test it. The original script recently removed the "objects_list" dictionary and replaced with a quick "if" based on the version of the Nvidia Driver. The result can be either "libnvcuvid.so" or "libnvidia-encode.so". I'm really new to Nix so I wasn't able to come up with a solution to determine the "object" automatically based on the driver version ([this is how it's done on the original script](https://github.com/keylase/nvidia-patch/blob/master/patch.sh#L288-L294)), so help on that would be awesome! (And pretty quick and easy for any experienced Nix user)

👍 1

aksiksi commented

2023-12-25 18:33:12 +00:00

(Migrated from github.com)

Thanks! This fix is working for me on 23.11 at latest nvenc-patch commit (forked this repo with the fix).

Thanks! This fix is working for me on 23.11 at latest `nvenc-patch` commit (forked this repo with the fix).

icewind1991 commented

2024-01-30 14:54:56 +00:00

(Migrated from github.com)

I have a NixOS machine with and nvidia gpu again so I was finally able to test this.