Re: [[PATCH v3] 4/6] media: Add variant info API/XML

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

 



On Thu, Nov 28, 2013 at 03:10:22PM +0000, Zeeshan Ali (Khattak) wrote:
> Add:
> 
> * ability to associate media to one or more variants of the OS in the
>   'media' XML node. For example:
> 
>   <os id="http://microsoft.com/win/7";>
>     <short-id>win7</short-id>
>     <name>Microsoft Windows 7</name>
>     ..
> 
>     <variant id="starter">
>       <name>Microsoft Windows 7 Starter</name>
>     </variant>
>     <variant id="home-basic">
>       <name>Microsoft Windows 7 Home Basic</name>
>     </variant>
>     <variant id="home-premium">
>       <name>Microsoft Windows 7 Home Premium</name>
>     </variant>
>     ..
> 
>     <media installer-reboots="2" arch="x86_64">
>       <variant id="home-premium"/>
>       <iso>
>         ..
>       </iso>
>     </media>
>   </os>
> 
> * API to query variants of the OS, media is associated with.
> ---
>  data/schemas/libosinfo.rng |  6 ++++++
>  osinfo/libosinfo.syms      |  2 ++
>  osinfo/osinfo_db.c         |  8 +++++++-
>  osinfo/osinfo_loader.c     | 18 ++++++++++++++++--
>  osinfo/osinfo_media.c      | 39 +++++++++++++++++++++++++++++++++++++++
>  osinfo/osinfo_media.h      |  3 +++
>  6 files changed, 73 insertions(+), 3 deletions(-)


> diff --git a/osinfo/osinfo_loader.c b/osinfo/osinfo_loader.c
> index affc49b..e9ce8e1 100644
> --- a/osinfo/osinfo_loader.c
> +++ b/osinfo/osinfo_loader.c
> @@ -859,7 +859,6 @@ static OsinfoMedia *osinfo_loader_media (OsinfoLoader *loader,
>      xmlChar *installer = xmlGetProp(root, BAD_CAST OSINFO_MEDIA_PROP_INSTALLER);
>      xmlChar *installer_reboots =
>              xmlGetProp(root, BAD_CAST OSINFO_MEDIA_PROP_INSTALLER_REBOOTS);
> -    xmlChar *variant = xmlGetProp(root, BAD_CAST OSINFO_MEDIA_PROP_VARIANT);

Ah, and now you remove the bogus variable from the previous patch...

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




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

  Powered by Linux