Bill Allombert:
> On Thu, Jul 23, 2015 at 10:42:59PM +0000, Patrick Schleizer wrote:
>> Bill Allombert:
>>> On Thu, Jul 23, 2015 at 12:46:25PM +0000, Patrick Schleizer wrote:
>>>> Package: menu
>>>> Severity: wishlist
>>>>
>>>> Dear maintainer!
>>>>
>>>> Please consider the following patch.
>>>>
>>>> It improves SU_TO_ROOT_X auto detection code. It prefers kdesudo
>>>> (because it's more modern, supports /etc/sudoers.d etc.) and gksudo over
>>>> gksu.
>>>
>>> Hello Patrick,
>>>
>>> Not all Debian systems are set up to allow users to use sudo.
>>> As far as I understand, kdesudo only work on such system,
>>> thus it cannot be prefered to kdesu.
>>>
>>> Cheers,
>>>
>>
>> Even if not disallowed. Even without any custom sudoers settings, this
>> patch would work? No disadvantages by it?
>>
>> kdesudo works on any system. "sudo apt-get install kdesudo", that's it.
>> No special settings required. And if one would not have it installed,
>> su-to-root would act as it does now. No difference then.
> 
> I just tested kdesudo and gksudo and they certainly do not work if the user is
> not allowed to use sudo, but is allowed to use su.
> 
> Are you seeing something different ?
> 
> Cheers,
> Bill.

You're right!

Covering that case in the updated patch.

Commit:
https://github.com/adrelanos/menu/commit/6626a5c6e7364c8cd26ae20dfb04962743fe9fbe

Branch:
https://github.com/adrelanos/menu/tree/kdesudo_gksudo_2

Pull request:
https://github.com/adrelanos/menu/pull/2

Cheers,
Patrick


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to