trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: November 2011

"Keep password" check box restored to kdesu dialog box

From: Darrell Anderson <humanreadable@...>
Date: Tue, 29 Nov 2011 19:00:05 -0800 (PST)
I submitted a patch to bug report 388 (http://bugs.pearsoncomputing.net/show_bug.cgi?id=388). The patch restores the visibility of the kdesu "Keep password" check box. In TDE, the check box is hard-coded invisible rather than being a function of the kdesu -t parameter as in 3.5.10.

The patch (kdebase/kdesu/kdesu/kdesu.cpp):

================================================
     // Try to exec the command with kdesud.
     bool keep = !args->isSet("n") && have_daemon;
-    bool terminal = true;
+    bool terminal = args->isSet("t");
     bool new_dcop = args->isSet("newdcop");
     bool withIgnoreButton = args->isSet("ignorebutton");
================================================

Although I suspect many will welcome the patch, there likely will be just as many who will fear being eaten in the dark by grues if this check box is visible.

I dislike the idea of those who want this check box visible to have to always patch the TDE sources in their build scripts.

Is there a way to test who wants this check box visible and those who don't so no build-time patching is required and everybody is happy?

I think the 'buntu family of distros don't use kdesu and instead use kdesudo. Perhaps this patch does not affect those distros. If that is the case then the discussion falls on everybody else.

Ideas?

Darrell