<p dir="ltr">I disagree on your opinion, waiting for additional and more competent comments...</p>
<p dir="ltr">D.</p>
<div class="gmail_quote">Il 25/feb/2015 05:56 "Zbigniew Jędrzejewski-Szmek" <<a href="mailto:zbyszek@in.waw.pl">zbyszek@in.waw.pl</a>> ha scritto:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Feb 20, 2015 at 11:01:18AM +0100, Daniele Medri wrote:<br>
> ---<br>
>  src/core/<a href="http://org.freedesktop.systemd1.policy.in.in" target="_blank">org.freedesktop.systemd1.policy.in.in</a> | 6 +++---<br>
>  src/login/<a href="http://org.freedesktop.login1.policy.in" target="_blank">org.freedesktop.login1.policy.in</a>     | 8 ++++----<br>
>  2 files changed, 7 insertions(+), 7 deletions(-)<br>
><br>
> diff --git a/src/core/<a href="http://org.freedesktop.systemd1.policy.in.in" target="_blank">org.freedesktop.systemd1.policy.in.in</a> b/src/core/<a href="http://org.freedesktop.systemd1.policy.in.in" target="_blank">org.freedesktop.systemd1.policy.in.in</a><br>
> index cc39a9e..11a27de 100644<br>
> --- a/src/core/<a href="http://org.freedesktop.systemd1.policy.in.in" target="_blank">org.freedesktop.systemd1.policy.in.in</a><br>
> +++ b/src/core/<a href="http://org.freedesktop.systemd1.policy.in.in" target="_blank">org.freedesktop.systemd1.policy.in.in</a><br>
> @@ -39,7 +39,7 @@<br>
><br>
>          <action id="org.freedesktop.systemd1.manage-unit-files"><br>
>                  <_description>Manage system service or unit files</_description><br>
> -                <_message>Authentication is required to manage system service or unit files.</_message><br>
> +                <_message>Authentication is required to handle system service or unit files.</_mes<br>
Doesn't seem to be an improvment.<br>
<br>
sage><br>
>                  <defaults><br>
>                          <allow_any>auth_admin</allow_any><br>
>                          <allow_inactive>auth_admin</allow_inactive><br>
> @@ -48,8 +48,8 @@<br>
>          </action><br>
><br>
>          <action id="org.freedesktop.systemd1.set-environment"><br>
> -                <_description>Set or unset system and service manager environment variables</_description><br>
> +                <_description>Configure system and service manager environment variables</_description><br>
This is less specific than the original. I'd say that it is worse.<br>
<br>
> -                <_message>Authentication is required to set or unset system and service manager environment variables.</_message><br>
> +                <_message>Authentication is required to handle system and service manager environment variables.</_message><br>
This one too. "handle" could mean anything. "set or unset" is maybe not pretty, but<br>
at least very clear.<br>
<br>
>                  <defaults><br>
>                          <allow_any>auth_admin</allow_any><br>
>                          <allow_inactive>auth_admin</allow_inactive><br>
> diff --git a/src/login/<a href="http://org.freedesktop.login1.policy.in" target="_blank">org.freedesktop.login1.policy.in</a> b/src/login/<a href="http://org.freedesktop.login1.policy.in" target="_blank">org.freedesktop.login1.policy.in</a><br>
> index 35bb390..906fea2 100644<br>
> --- a/src/login/<a href="http://org.freedesktop.login1.policy.in" target="_blank">org.freedesktop.login1.policy.in</a><br>
> +++ b/src/login/<a href="http://org.freedesktop.login1.policy.in" target="_blank">org.freedesktop.login1.policy.in</a><br>
> @@ -271,8 +271,8 @@<br>
>          </action><br>
><br>
>          <action id="org.freedesktop.login1.manage"><br>
> -                <_description>Manager active sessions, users and seats</_description><br>
> +                <_description>Active sessions, users and seats management</_description><br>
I think this was just a typo, extra "r", and the original wording is OK.<br>
This was actually already fixed in git.<br>
<br>
> -                <_message>Authentication is required for managing active sessions, users and seats.</_message><br>
> +                <_message>Authentication is required to handle active sessions, users and seats.</_message><br>
>                  <defaults><br>
>                          <allow_any>auth_admin_keep</allow_any><br>
>                          <allow_inactive>auth_admin_keep</allow_inactive><br>
> @@ -281,8 +281,8 @@<br>
>          </action><br>
><br>
>          <action id="org.freedesktop.login1.lock-sessions"><br>
> -                <_description>Lock or unlock active sessions</_description><br>
> -                <_message>Authentication is required for locking or unlocking active sessions.</_message><br>
> +                <_description>Active sessions management</_description><br>
> +                <_message>Authentication is required to handle active sessions.</_message><br>
Less specific.<br>
<br>
But we should say "to lock or unlock". I'll fix that.<br>
<br>
All in all, I think there's nothing to fix here.<br>
<br>
Zbyszek<br>
</blockquote></div>