Skip to content

Conversation

@joebonrichie
Copy link
Contributor

@joebonrichie joebonrichie commented Dec 1, 2025

Summary

wheel is the default and sudo is a debianism from the solus 2 days. Requires us to patch things so they work as expected but invetiably we've missed some things over the years causing some functionality to not work as expected.

We will not yet remove admin users from the sudo group until we are confident we've found everything

Resolves #299, #7169.

Test Plan

Add packages to local repo, update and reboot

  • User is now part of wheel group getent group wheel
  • flatpak history now works without sudo requried if part of the wheel group
  • journalctl shows all the journal without sudo required if part of the wheel group

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

@github-project-automation github-project-automation bot moved this to Triage in Solus Dec 1, 2025
@joebonrichie joebonrichie changed the title Transition to wheel from sudo as the privilaged group Transition to wheel from sudo as the privileged group Dec 1, 2025
@joebonrichie
Copy link
Contributor Author

tentatively marking as ready for review

@joebonrichie joebonrichie marked this pull request as ready for review January 6, 2026 13:11
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

I found some more packages that need to be modified:

  • gvfs
  • gnome-control-center
  • cups
  • blueman
  • flatpak
  • budgie-control-center
  • bolt

@EbonJaeger EbonJaeger added the Needs Rebase Needs rebasing by PR author label Jan 13, 2026
@EbonJaeger
Copy link
Member

Also needs a rebase for qol-assist. Sorry!

@joebonrichie
Copy link
Contributor Author

I found some more packages that need to be modified:

* `gvfs`

* `gnome-control-center`

* `cups`

* `blueman`

* `flatpak`

* `budgie-control-center`

* `bolt`

Thanks, added. I'd like to do another couple of online & offline upgrade tests with these to ensure nothing blows up due to the group changing underneath it but I think it should be fine.

@malfisya malfisya added Topic: Sync Notes This PR/Issue can be highlighted in sync notes and removed Needs Rebase Needs rebasing by PR author labels Jan 17, 2026
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! At the very least, I want to hold off on merging this until the scheduled package server maintenance is completed. Do you want to do some more testing as well before this goes in?

@joebonrichie
Copy link
Contributor Author

LGTM, thanks! At the very least, I want to hold off on merging this until the scheduled package server maintenance is completed. Do you want to do some more testing as well before this goes in?

Only if you find some time you'll willing to do it rather than it being a burden

@EbonJaeger EbonJaeger merged commit 1904e56 into main Jan 17, 2026
1 check passed
@EbonJaeger EbonJaeger deleted the wheel branch January 17, 2026 21:01
@github-project-automation github-project-automation bot moved this from Triage to Done in Solus Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Topic: Sync Notes This PR/Issue can be highlighted in sync notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Transition the default sudo group from sudo to wheel (T10378)

4 participants