Create Task
Maniphest T12165

KDESu
Open, Needs TriagePublic
Actions

Related Objects
Search...

StatusAssignedTask
 OpenNoneT12165 KDESu
 OpenNoneT12166 KDESu: Close fds manually instead of relying on KService
fabiank created this task.Nov 23 2019, 4:26 PM
nicolasfella moved this task from Backlog to Metatasks on the KF6 board.Nov 25 2019, 9:26 PM
stikonas added a subscriber: stikonas.Dec 27 2019, 11:57 AM

Do we still expect to have any users of kdesu? Most apps were probably ported to polkit already.

dfaure added a subscriber: dfaure.Jan 1 2020, 10:52 AM

Good question. https://lxr.kde.org/search?_filestring=&_string=KF5%3A%3ASu says konqueror/plugins/shellcmdplugin and workspace/kde-cli-tools/kdesu.

I'm fine with SU support being removed from the shellcmdplugin, but we definitely need the kdesu executable, useful for end users, to remain. At least this means the framework could be merged into the executable in kde-cli-tools.

stikonas added a comment.Jan 1 2020, 12:37 PM

There is also Cervisia that uses KDESu::PtyProcess.

dfaure added a comment.Jan 1 2020, 5:31 PM

Do you think it could use polkit? Or is this about an embedded root terminal or something?

stikonas added a comment.Jan 1 2020, 9:53 PM
In T12165#216097, @dfaure wrote:

Do you think it could use polkit? Or is this about an embedded root terminal or something?

I think it's just using some kdesu functions to communicate with command line programs (cvs?) and nothing to do with root (but I only had a brief look at it).

nicolasfella added a subscriber: nicolasfella.Nov 23 2021, 5:02 PM

From today's meeting: KDESu::PtyProcess seems to predate KPtyProcess and that's likely the reason the former is used. Unless we are missing something and cvs(?) actually needs to be run as root