[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [RFR] templates://kdesudo/{kdesudo.templates}



Christian Perrier wrote:
> Your review should be sent as an answer to this mail.

>  Template: kdesudo/kdesu         
[...]
>  _Description: Use kdesudo as kdesu?

To nitpick, the question isn't whether I want to use kdesudo as
kdesu (i.e. as a front-end to /bin/su), the question is whether I
want to use kdesudo as the default KDE SUIDifier.  Say:

   _Description: Use KdeSudo in place of KdeSu?

(In fact upstream would make that "in place of KDE su", but let's
standardise on the KdeSudo style.)

> + Kdesudo is command-line compatible with kdesu.

This seems misleading to me, since kdesu and kdesudo aren't for use
on the commandline at all; as far as keyboard interaction is
concerned, they're entirely incompatible, since they require 
different typed-in passwords.  However, it should be safe to say:

    KdeSudo is a drop-in replacement for KdeSu.

> + Please choose whether you want to use kdesudo as a drop-in replacement
> + for kdesu.

I might be setting this up because the boss says I have to!  So I'd
prefer to see something like:

    Please choose whether KdeSudo should be the default front-end for
    gaining elevated privileges under KDE.

(Explaining in passing what it's actually for.)

>  Package: kdesudo
[...]
>  Description: sudo frontend for KDE
>   KdeSudo is a graphical frontend for the sudo utility.
> + It allows to run programs with another user's privileges by entering
> + the current user's password.

That construction isn't allowed.  And KdeSudo doesn't do anything by
entering a password.  Besides, it isn't the front-end that provides
this functionality, it's sudo!

After a lot of wrestling with third person possessives the most
natural way I can find to say it is:

    KdeSudo is a graphical frontend for the sudo utility, which allows
    users to run programs as root (or another user) by giving their own
    password.

-- 
JBR	with qualifications in linguistics, experience as a Debian
	sysadmin, and probably no clue about this particular package
--- ../kdesudo.old/debian/kdesudo.templates	2007-11-29 07:50:58.000000000 +0000
+++ debian/kdesudo.templates	2007-12-08 12:10:47.000000000 +0000
@@ -1,6 +1,8 @@
 Template: kdesudo/kdesu
 Type: boolean
 Default: false
-_Description: Use kdesudo as kdesu?
- Kdesudo is command line compatible with kdesu and can be used as a drop-in 
- replacement. There is a big chance that you want kdesudo used as kdesu.
+_Description: Use KdeSudo in place of KdeSu?
+ KdeSudo is a drop-in replacement for KdeSu.
+ .
+ Please choose whether KdeSudo should be the default front-end for
+ gaining elevated privileges under KDE.
--- ../kdesudo.old/debian/control	2007-11-29 07:50:58.000000000 +0000
+++ debian/control	2007-12-08 12:48:50.000000000 +0000
@@ -10,5 +10,6 @@
 Architecture: any
 Depends: ${shlibs:Depends}, ${misc:Depends}, sudo
 Description: sudo frontend for KDE
- KdeSudo is a graphical frontend for the sudo utility.
- It allows you to run programs as another user by entering your password.
+ KdeSudo is a graphical frontend for the sudo utility, which allows
+ users to run programs as root (or another user) by giving their own
+ password.
Template: kdesudo/kdesu
Type: boolean
Default: false
_Description: Use KdeSudo in place of KdeSu?
 KdeSudo is a drop-in replacement for KdeSu.
 .
 Please choose whether KdeSudo should be the default front-end for
 gaining elevated privileges under KDE.
Source: kdesudo
Section: kde
Priority: optional
Maintainer: Sune Vuorela <debian@pusling.com>
Build-Depends: debhelper (>> 5), cdbs, kdelibs4-dev, docbook2x
Standards-Version: 3.7.2
Homepage: https://launchpad.net/kdesudo

Package: kdesudo
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}, sudo
Description: sudo frontend for KDE
 KdeSudo is a graphical frontend for the sudo utility, which allows
 users to run programs as root (or another user) by giving their own
 password.

Reply to: