Skip to content
This repository has been archived by the owner on Feb 17, 2024. It is now read-only.

NVIDIA 495.44 Linux Driver Release and replace witch with command -v #59

Closed
wants to merge 2 commits into from
Closed

Conversation

Neo2SHYAlien
Copy link
Contributor

  • Drop "Kepler" GPU's
  • GBM Support
  • indicator for Resizable BAR (ReBAR) support
  • requires at least Linux 3.10
/usr/bin/which: this version of `which' is deprecated; use `command -v' in scripts instead.

/usr/bin/which: this version of `which' is deprecated; use `command -v'
in scripts instead.
* Drop "Kepler" GPU's
* GBM Support
* indicator for Resizable BAR (ReBAR) support
* requires at least Linux 3.10
Copy link
Owner

@smxi smxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder about this deprecation, where is that coming from? That's an odd thing to deprecate, it's not like it has any real logic behind it that will suffer bit rot, in fact, they could just map the command to something else behind the scenes and nobody would know the difference. For this one, going to change it to type -p, which is used in other tools here, actually never seen command -v before. Better to keep consistent across everything, just switch it all to type -p, some of it already used that, don't remember, easier.

@smxi smxi closed this Nov 7, 2021
Copy link
Owner

@smxi smxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making these in two requests, makes it easier. Moved all the smxi, svmi, and sgfxi to use type -p, that must be new in sid or something re the deprecation.

Copy link
Owner

@smxi smxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note, I'm just going to do this one manually, sorry, I thought these were two requests, don't want to deal with git stuff so easier to just close the request and do your driver patch manually.

Copy link
Owner

@smxi smxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note the commit I did, this driver only supports 3.10 and newer kernels, so the kernel support tests had to be tweaked a bit after the search and replace, but next time should be fine fora general search/replace update.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants