kdesu default super-user-command

[Thomas Friedrichsmeier]

Hi again,

On Wednesday 21 November 2007, Thomas Friedrichsmeier wrote:
> In KDE3, revision 571012 changes the default super-user-command to "su",
> back from "sudo" (added less than 24 hours before that). Revision 592501
> makes the default a build-time configuration option. Neither seems to have
> been forward-ported to KDE 4 (where the default is hard-coded as "sudo"). I
> could not find anything about this on the mailing lists. So: Is this a
> deliberate decision or an oversight?

no reply so far, so I'll be more specific:

I'd like to apply the attached patch (kdelibs/kdesu/). This reverts the 
default super user command in kdesu back to "su" from "sudo", and adds a 
cmake switch to change the default to "sudo". Reasons:
- "su" was the default in KDE3
- it's what the name "kdesu" suggests
- sudo requires configuration (/etc/sudoers) to work which is likely done by 
the distro. But then the distro can easily use the cmake switch. su should 
work with kdesu out of the box.
- the revision history suggests that defaulting to "sudo" in KDE 4 is likely 
to have been an oversight, not a deliberate decision. Correct me, if this 
impression is wrong.

Follow-up question: CMake variables like this should likely be documented in 
some central place. Where?

Regards
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdesu_default_su_command.patch
Type: text/x-diff
Size: 1446 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071122/9e36b0e4/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071122/9e36b0e4/attachment.sig>