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

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

 



On 10/13/2012 08:28 AM, Cole Robinson wrote:
> - 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.

ACK; if Dan agrees to the relicense to LGPLv2+, then that could be
squashed in as well.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]