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

[libvirt] [PATCH] Fix PolicyKit.conf example docs



The current docs showing a possible PolicyKit.conf with the example:

  <match action="org.libvirt.unix.manage" user="fred">
    <return result="yes"/>
  </match>

With PolicyKit-0.6-2.fc8, polkit-config-file-validate complains
about this format. 'man PolicyKit.conf' gives a similar example
with the format:

  <match action="org.libvirt.unix.manage">
    <match user="fred">
      <return result="yes"/>
    </match>
  </match>

This doesn't cause any complaints and works. The attached patch updates
the docs with this format.

Thanks,
Cole
diff --git a/docs/auth.html.in b/docs/auth.html.in
index 7340360..04b1210 100644
--- a/docs/auth.html.in
+++ b/docs/auth.html.in
@@ -77,11 +77,15 @@ while requiring <code>joe</code> to authenticate with the admin password,
 would require adding the following snippet to <code>PolicyKit.conf</code>.
 </p>
     <pre>
-  &lt;match action="org.libvirt.unix.manage" user="fred"&gt;
-    &lt;return result="yes"/&gt;
+  &lt;match action="org.libvirt.unix.manage"&gt;
+    &lt;match user="fred"&gt;
+      &lt;return result="yes"/&gt;
+    &lt;/match&gt;
   &lt;/match&gt;
-  &lt;match action="org.libvirt.unix.manage" user="joe"&gt;
-    &lt;return result="auth_admin"/&gt;
+  &lt;match action="org.libvirt.unix.manage"&gt;
+    &lt;match user="joe"&gt;
+      &lt;return result="auth_admin"/&gt;
+    &lt;/match&gt;
   &lt;/match&gt;
 </pre>
     <h3><a name="ACL_server_username">Username/password auth</a></h3>

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