Re: [PATCH python] fix libvirt headers list

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

 



On 10/28/2014 10:58 AM, Dmitry Guryanov wrote:
> Since libvirt.h has been splitted out, generator.py

s/splitted/split/

> should be fixed accordingly. So add full list of header
> files.
> 

Does this still work when targetting older libvirt prior to the header
split?  If so, then ACK.

>  
> +libvirt_headers = ["libvirt",
> +                   "libvirt-domain",
> +                   "libvirt-domain-snapshot",
> +                   "libvirt-event",
> +                   "libvirt-host",
> +                   "libvirt-interface",
> +                   "libvirt-network",
> +                   "libvirt-nodedev",
> +                   "libvirt-nwfilter",
> +                   "libvirt-secret",
> +                   "libvirt-storage",
> +                   "libvirt-stream",]

Might be worth putting the ] on its own line; then, if we add new files
in the future, it will be a one line addition instead of having to
modify an existing line.

-- 
Eric Blake   eblake redhat com    +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]