FS#65761 - [sway] polkit should be added as dependency
Attached to Project:
Community Packages
Opened by Faalagorn (Faalagorn) - Monday, 09 March 2020, 22:40 GMT
Last edited by Brett Cornwall (ainola) - Friday, 04 June 2021, 05:05 GMT
Opened by Faalagorn (Faalagorn) - Monday, 09 March 2020, 22:40 GMT
Last edited by Brett Cornwall (ainola) - Friday, 04 June 2021, 05:05 GMT
|
Details
Both sway and cage require polkit be added as a dependency
in order to be usable from a base arch installation. Without
it both DEs would fail to launch, citing inability to gain
CAP_SYS_ADMIN in order to become a DRM master.
Original Author Description: It seems that colord is need to launch sway. It took me a while to figure why sway isn't launching on a fresh system and installing weston helped. It turned out colord was needed dependency for sway. Steps to reproduce: 1. Install a fresh system 2. Install sway 3. Try to launch it 4. Sway will not launch 5. Install colord 6. Sway will launch |
This task depends upon
Closed by Brett Cornwall (ainola)
Friday, 04 June 2021, 05:05 GMT
Reason for closing: Fixed
Additional comments about closing: Cool, sounds like we'll keep things as it is then.
Friday, 04 June 2021, 05:05 GMT
Reason for closing: Fixed
Additional comments about closing: Cool, sounds like we'll keep things as it is then.
Note that I don't know anything about polkit, so I might be wrong and polkit might actually be contributing to all this, but I'd be very surprised.
Then if you really think polkit is required, it'd be interesting to know what polkit definition comes into play.
Thanks!
Maintainers, any issue with removing the polkit dep and installing sway as setuid?
[1] https://github.com/swaywm/sway/issues/5260#issuecomment-619542107
Can this be consolidated with upstream?
Well, then nothing should indeed be done from our side. We use systemd and logind, hence we do *not* use setuid.
The systemd package already lists polkit as an optional dependency, which should be enough. However, we can add Christian Hesse to this ticket and ponder over whether it would make sense to promote it to a full dependency.
And since you added Christian: I find the description of systemd's optional dependency on polkit to be somewhat misleading in light of this particular case, since polkit is needed for something that is not "administration".
Regarding the "unfounded" statements, please see the link in the same post.
I'm fine with continuing with the status quo but I wanted to make sure that there was concensus.
Cf https://github.com/swaywm/sway/issues/3668 for another byte of context.
For the record, another option here could be to make polkit an optional dependency of sway, with an unequivocal description. But you're probably going to see that as "broken-by-default", and I won't argue with that.