[pkg-wine-party] Bug#848839: AppStream metadata for Wine

[Jens Reyer]

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>