[PATCH 2/2] Tweak comments in the policykit rules file

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

 



- Add the XML header so vim gives us syntax highlighting
- polkit-policy-file-validate hasn't existed for 3 years
- Permissions comment was not accurate
---
 daemon/libvirtd.policy.in | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/daemon/libvirtd.policy.in b/daemon/libvirtd.policy.in
index 45b0d79..000c17e 100644
--- a/daemon/libvirtd.policy.in
+++ b/daemon/libvirtd.policy.in
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE policyconfig PUBLIC
  "-//freedesktop//DTD PolicyKit Policy Configuration 1.0//EN"
  "http://www.freedesktop.org/standards/PolicyKit/1.0/policyconfig.dtd";>
@@ -9,10 +10,6 @@ Copyright (c) 2007 Daniel P. Berrange <berrange redhat com>
 
 libvirt is licensed to you under the GNU Lesser General Public License
 version 2. See COPYING for details.
-
-NOTE: If you make changes to this file, make sure to validate the file
-using the polkit-policy-file-validate(1) tool. Changes made to this
-file are instantly applied.
 -->
 
 <policyconfig>
@@ -32,8 +29,8 @@ file are instantly applied.
       <description>Manage local virtualized systems</description>
       <message>System policy prevents management of local virtualized systems</message>
       <defaults>
-        <!-- Only a program in the active host session can use libvirt in
-             read-write mode for management, and we require user password -->
+        <!-- Any program can use libvirt in read/write mode if they
+             provide the root password -->
         <allow_any>auth_admin</allow_any>
         <allow_inactive>auth_admin</allow_inactive>
         <allow_active>@authaction@</allow_active>
-- 
1.7.11.4

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]