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

Add ubuntu drivers backend #132

Closed
wants to merge 17 commits into from
Closed

Add ubuntu drivers backend #132

wants to merge 17 commits into from

Conversation

leolost2605
Copy link
Member

@leolost2605 leolost2605 commented Feb 7, 2024

Fixes #140

Should now be on par with what appcenter does. The only open question for me is this:

  • Do we still need to install kernel headers like it's currently done in appcenter? I've recently seen a PR about adding them by default so I don't know?

@danirabbit
Copy link
Member

@leolost2605 Yeah we've had people, especially with the nvidia driver, who want to switch to another version of the driver. It looks like Ubuntu lists them all as radios: https://linuxconfig.org/wp-content/uploads/2019/12/02-how-to-install-the-nvidia-drivers-on-ubuntu-20-04-focal-fossa-linux.png

@leolost2605
Copy link
Member Author

leolost2605 commented Feb 18, 2024

Ok I finally got around to testing this at least with nvidia drivers and it should work now.

Some things to note:

  • You currently can't remove drivers because that would require running autoremove and since it's currently the same in appcenter this is left to follow up PRs
  • With nvidia drivers, installing another one while one is already installed automatically removes the other one. They aren't shown with radio buttons because that would require either assuming all drivers with nvidia in their name are for the same device or redoing quite a lot of this and I kind of don't want to do either since I don't have a lot of time ATM. Also it is on par with appcenter and I would say even a bit more understandable for the user.
  • Add driver view switchboard-plug-about#300 is the corresponding UI PR

@leolost2605 leolost2605 marked this pull request as ready for review February 18, 2024 00:45
@leolost2605 leolost2605 requested a review from a team February 18, 2024 00:45
@leolost2605
Copy link
Member Author

See #141 for v2 of this

@leolost2605 leolost2605 marked this pull request as draft February 18, 2024 22:30
@danirabbit
Copy link
Member

Closing since we merged in the other version

@danirabbit danirabbit closed this May 30, 2024
@danirabbit danirabbit deleted the drivers branch May 30, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Handle system driver management
2 participants