Re: [v2] Add an optional 'snapshot' tag to OS entries

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

 



On Mon, Nov 04, 2013 at 04:11:20PM +0000, Zeeshan Ali (Khattak) wrote:
> It wasn't previously clear to me that you want to help. Now that it
> is, here is the my attempt to fix this issue:
> 
> https://gitorious.org/libosinfo/libosinfo/commit/77531e31ec579d5ce6857f09e983325bb6b56d5c
> 
> As you can see, I have tried to tell the loader not to ignore empty
> elements. However, for some reason unknown to me, this break things
> completely as libosinfo to loads OS entries where there is no name
> provided.

What happens is that string(./foo) in xpath returns "" even for
non-existing nodes. If you combine this with the code to load localized
values (string(./foo[lang('XX_xx')])), then this means we'll get "" for any
non-localized attribute. The patch at
http://paste.fedoraproject.org/51766/66649413/ appears to be working, I
want to play a bit more with it before sending it.

Christophe

Attachment: pgpJwdcjEN2KG.pgp
Description: PGP signature

_______________________________________________
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