Bug 15088 - lxqt - pcmanfm-qt cannot find the kdesu/gksu path in open as root menu
Summary: lxqt - pcmanfm-qt cannot find the kdesu/gksu path in open as root menu
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Florian Hubold
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-19 20:02 CET by Dimitrios Glentadakis
Modified: 2015-01-21 19:54 CET (History)
4 users (show)

See Also:
Source RPM: pcmanfm-qt-0.8.0-3.mga5.src.rpm
CVE:
Status comment:


Attachments

Description Dimitrios Glentadakis 2015-01-19 20:02:46 CET
Description of problem: 
In pcmanfm-qt file manager, when i choose from the menu: Tools>Open as Root, i have the message that the kdesu/gksu couldn't be find

To workarround it i linked the /usr/lib64/kde4/libexec/kdesu to my personal folder (/home/user/bin)

Reproducible: 

Steps to Reproduce:
Comment 1 Florian Hubold 2015-01-19 21:03:34 CET
Are you sure that it looks for kdesu? I can't find any reference in pcmanfm-qt sources that refers to kdesu, only to gksu.

Anyway, seems we cannot replace it simply by pkexec, as pkexec does not support any arguments to the program called (like the path where pcmanfm should start) AND it also requires additional configuration to allow running X applications which is also not recommended per the man page.

So the question is mainly - do we want pcmanfm-qt to require either gksu-polkit for gksu or kdebase4-runtime for kdesu ? gksu requires ~500KB and 3 packages based on minimal LXQt install, kdebase4-runtime requires 6MB. Also for kdesu it needs a slight adaption, as that is located in %_libdir and not in /usr/bin.

@Dimitrios: WDYT?

Status: NEW => ASSIGNED
CC: (none) => doktor5000
Hardware: x86_64 => All

Comment 2 Florian Hubold 2015-01-19 21:06:08 CET
Major difference - when run with gksu, no icons are shown anymore. With kdesu it works normally.
Comment 3 Florian Hubold 2015-01-19 21:18:21 CET
I'd say kdebase4-runtime fits better. First it's also based on QT, second it provides another window manager that is supported by LXQt, and third it provides kdesu. And running pcmanfm via kdesu shows icons, whereas with gksu it doesn't.

Any objections?
Comment 4 Dimitrios Glentadakis 2015-01-19 21:43:20 CET
I agree to use a qt dialogue and if it works better the kdesu it is ok (i use some kde applications also)

I searched a little and it seems that we miss the xdg-su command ?
Comment 5 Florian Hubold 2015-01-20 21:44:46 CET
For pcmanfm-qt, I've added support for kdesu now and submitted pcmanfm-qt-0.8.0-4.mga5 - please reopen if it's not working for you.


(In reply to Dimitrios Glentadakis from comment #4)
> I searched a little and it seems that we miss the xdg-su command ?

Yep, seems so - short search shows most distros have added that years ago. strange thing is the only released xdg-utils version that contains it is http://portland.freedesktop.org/xdg-utils-1.0beta1/ Feel free to open another bugreport for that and add it :)

Seems like Atilla has added it to his github site, maybe you want to discuss this with him? https://github.com/tarakbumba/xdg-su/blob/master/README

Status: ASSIGNED => RESOLVED
CC: (none) => tarakbumba
Resolution: (none) => FIXED

Comment 6 Alex Loginov 2015-01-20 22:17:19 CET
Why not beesu?

CC: (none) => loginov_alex

Comment 7 Sander Lepik 2015-01-21 12:11:23 CET
My comments from IRC:
<sander85> doktor5000: http://svnweb.mageia.org/packages/cauldron/pcmanfm-qt/current/SPECS/pcmanfm-qt.spec?r1=803666&r2=811608 please revert the requires
<sander85> doktor5000: it won't play nice with plasma 5 if you pull half the kde4 in.. it can recommend this package but it should be possible to install w/o it!

Status: RESOLVED => REOPENED
CC: (none) => mageia
Resolution: FIXED => (none)

Comment 8 Atilla ÖNTAŞ 2015-01-21 14:37:30 CET
(In reply to Florian Hubold from comment #5)
> For pcmanfm-qt, I've added support for kdesu now and submitted
> pcmanfm-qt-0.8.0-4.mga5 - please reopen if it's not working for you.
> 
> 
> (In reply to Dimitrios Glentadakis from comment #4)
> > I searched a little and it seems that we miss the xdg-su command ?
> 
> Yep, seems so - short search shows most distros have added that years ago.
> strange thing is the only released xdg-utils version that contains it is
> http://portland.freedesktop.org/xdg-utils-1.0beta1/ Feel free to open
> another bugreport for that and add it :)
> 
> Seems like Atilla has added it to his github site, maybe you want to discuss
> this with him? https://github.com/tarakbumba/xdg-su/blob/master/README

Hi. I think xdg-su is not suitable for this purpose. You can see that xdg-su searches for other gui su helpers like kdesu, gksu, ktuss etc. and when found it uses that utility. So, you should also install kdesu, beesu etc. with xdg-su. Only exception to this is if no su frontend utility found in system; xdg-su opens a terminal window and asks for su password to run given command as root.

I think using beesu or if packaged ktsuss for this purpose is rather simpler and efficient. My 2 cents.
Comment 9 Florian Hubold 2015-01-21 19:54:39 CET
FWIW, beesu exhibits the same problem as gksu, no icons shown. Also it's based on deprecated usermode/consolehelper which is unmaintained upstream since several years.


(In reply to Sander Lepik from comment #7)

> <sander85> doktor5000: it won't play nice with plasma 5 if you pull half the
> kde4 in.. it can recommend this package but it should be possible to install
> w/o it!

Well, plasma5 is not meant for productive usage by endusers, and it should be possible to have it in parallel with KDE4 - hence this would only be an ugly workaround.

Sure, I can use recommends instead, but for those using --no-recommends the function "run as root" will break, and they won't be able to use kwin which is supported by LXQt. But if you think that is a better option, only to allow to install it in parallel with plasma 5 ...


Changed with pcmanfm-qt-0.8.0-5.mga5

Status: REOPENED => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.