[pkg-wine-party] Bug#848839: AppStream metadata for Wine
Jens Reyer
jre.winesim at gmail.com
Fri Jan 13 16:34:40 UTC 2017
Thanks a lot Matthias!
On 01/12/2017 07:40 PM, Matthias Klumpp wrote:
>> There is a wine.desktop, but for other reasons we only ship it as an
>> example. Still, other distros probably install it. However that
>> .desktop file has "NoDisplay=true" so afaik it wouldn't be used for
>> AppStream anyway.
>
> That's not necessarily the case - if a metainfo file is provided, a
> NoDisplay field is ignored.
Did you use "metainfo" generally here, or specifically for foo.metainfo.xml?
> "desktop" btw is an outdated name, to describe applications you can
> pick the component types "desktop-application" and
> "console-application"
Thanks, changed.
<component type="desktop"> is still widespread in the documentation
(tell me if I should file separate bugreports/submit patches somewhere):
https://www.freedesktop.org/software/appstream/docs/sect-Metadata-Application.html
"Note that the XML root must have the type property set to desktop"
^^^^^^^
"All tags defined in the generic component specification are valid for
desktop application components as well."
--> Suggestion: add "(and vice versa)"
https://www.freedesktop.org/software/appstream/docs/chap-Quickstart.html#sect-Quickstart-DesktopApps
"<component type="desktop">"
^^^^^^^
https://wiki.debian.org/AppStream/Guidelines
"you can tell by the XML root-node having a type="desktop" attribute"
^^^^^^^
With appstream 0.10.5-1:
$ appstream-util appdata-from-desktop foo.desktop foo.appdata.xml"
--> type="desktop"
> For the example file:
> The validation fails with:
>
> Could not parse XML data: Entity: line 2: parser error : Start tag expected, '<'
> not found
> <!-- Copyright 2017 Jens Reyer <jre.winesim at gmail.com> -->
> ^
>
> I assume this is due to the < being some other character, because
> rewriting the header worked well.
Ouch, thanks! These were 'ZERO WIDTH SPACE' (U+200B) characters. I had
seen "appstream-util validate" complaining, but had assumed I did the
test wrongly.
This was based on the example file from
https://www.freedesktop.org/software/appstream/docs/chap-Quickstart.html,
copied in Debian from firefox to emacs. If I copy to vim I indeed see
them. So I guess the homepage needs to be fixed.
> The icon types "cached", "remote" and "local" are not allowed in
> metainfo files (reminds me to add a validator test for that), only
> "stock" is fine.
Sounds as if you are referring explicitly to "foo.metainfo.xml" files
here. Should I use metainfo.xml or appdata.xml?
https://www.freedesktop.org/software/appstream/docs/chap-CollectionData.html
says "stock icons are loaded from stock."
I don't understand what this exactly means. Where is this stock, and how
is it created/what does it contain? Do I as packager have any direct
influence on what it contains?
Even if I address all other issues and rename to metainfo.xml I still get:
$ appstream-util validate-relax org.winehq.wine.development.metainfo.xml
org.winehq.wine.development.metainfo.xml: FAILED:
• markup-invalid : <id> does not have correct extension for kind
Validation of files failed
Is this critical? Can I ignore it or do I need to use type "generic" (I
want to see Wine in Gnome Software Center)?
What do you use to validate?
> Otherwise the file looks fine, a screenshot might be nice though.
Thanks. I'll discuss screenshots and generic release info with upstream,
once I submit it there.
> (Edited file is attached)
Thanks again!
> P.S: Let me know when an updated Wine is uploaded, this will be the
> only app I know which does not use the metainfo file to augment a
> .desktop file, and I am curious to see if the file is handled
> correctly.
Will do, maybe later today.
Greets!
jre
-------------- next part --------------
A non-text attachment was scrubbed...
Name: org.winehq.wine.development.metainfo.xml
Type: text/xml
Size: 1678 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-wine-party/attachments/20170113/14158b2f/attachment.xml>
More information about the pkg-wine-party
mailing list