Bug 1185301

gnome-builder.x86_64: E: invalid-appdata-file /usr/share/appdata/org.gnome.Builder.appdata.xml

$ appstream-util validate /usr/share/appdata/org.gnome.Builder.appdata.xml 
/usr/share/appdata/org.gnome.Builder.appdata.xml: FAILED:
• style-invalid         : Not enough <p> content before <ul>

minor:
gnome-builder.spec:6: W: mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 1)

Thanks for the review! The AppData guidelines suggest that "validate-relax" is sufficient (which is included in the spec, and passes locally), although it would of course be nice if there were a few more paragraphs. I fixed the tabs-and-spaces warning and reuploaded the spec and srpm in place.

License tag is incorrect, if I checked correctly it should be:

    License:       GPLv3+ and GPLv2+ and LGPLv3+ and LGPLv2+ and MIT and CC-BY-SA and CC0

You could add a comment just above the license tag explaining what makes it so complex, like I had in the spec file for my Copr:

# Most of GNOME Builder is licensed under the GPLv3+.
#
# The following files are licensed under the GPLv2+:
#     - src/dialogs/gb-close-confirmation-dialog.c
#     - src/dialogs/gb-close-confirmation-dialog.h
#     - src/gedit/gedit-menu-stack-switcher.c
#     - src/gedit/gedit-menu-stack-switcher.h
#
# The following files are licensed under the LGPLv2+:
#     - data/styles/builder-dark.xml (seems like an error, the header speaks about GtkSourceView)
#     - src/animation/gb-frame-source.c
#     - src/animation/gb-frame-source.h
#     - src/fuzzy/fuzzy.c
#     - src/fuzzy/fuzzy.h
#     - src/gd/gd-tagged-entry.c
#     - src/gd/gd-tagged-entry.h
#     - src/gedit/gedit-close-button.c
#     - src/gedit/gedit-close-button.h
#     - src/nautilus/nautilus-floating-bar.c
#     - src/nautilus/nautilus-floating-bar.h
#     - src/trie/trie.c
#     - src/trie/trie.h
#     - src/util/gb-cairo.c
#     - src/util/gb-cairo.h
#     - src/util/gb-glib.h
#     - src/util/gb-gtk.c
#     - src/util/gb-gtk.h
#
# The following files are licensed under the LGPLv3+:
#     - src/auto-indent/gb-source-auto-indenter.c
#     - src/auto-indent/gb-source-auto-indenter.h
#     - src/auto-indent/gb-source-auto-indenter-c.c
#     - src/auto-indent/gb-source-auto-indenter-c.h
#     - src/auto-indent/gb-source-auto-indenter-python.c
#     - src/auto-indent/gb-source-auto-indenter-python.h
#     - src/auto-indent/gb-source-auto-indenter-xml.c
#     - src/auto-indent/gb-source-auto-indenter-xml.h
#
# The following files are MIT licensed:
#     - src/resources/css/markdown.css
#     - src/resources/js/marked.js
#
# The following files are licensed under the CC-BY-SA license:
#     - data/icons/
#
# The following files are licensed under the CC0 license:
#     - data/org.gnome.Builder.appdata.xml
#     - data/html-preview.png

Other than that, please use %license for the COPYING file, rather than %doc.

Finally, that #VCS comment seems wrong, unless you're packaging cheese again. :)

(In reply to Mathieu Bridon from comment #3)
> License tag is incorrect, if I checked correctly it should be:
>…

Thanks for that! I would normally trim this down a bit to the effective license of the binary, which in this case would allow dropping of the LGPLv2+ and GPLv2+ bits (as all the files under those licenses are built into a binary with GPLv3+ code, and effectively GPLv3+). However, this probably needs too much careful examination of licenses to maintain well.

In any case, fixed now.

> Other than that, please use %license for the COPYING file, rather than %doc.

Cool, this was not part of the licensing guidelines the last time I checked, so fixed too.

> Finally, that #VCS comment seems wrong, unless you're packaging cheese
> again. :)

Whoops! Fixed now.

Scratch build, including the changes, and a 32-bit build fix from upstream (also reuploaded in place):

http://koji.fedoraproject.org/koji/taskinfo?taskID=8707142

directory /usr/share/icons/hicolor/scalable/autocomplete/ without known owner


E: invalid-appdata-file /usr/share/appdata/org.gnome.Builder.appdata.xml
interesting.

(In reply to Igor Gnatenko from comment #5)
> directory /usr/share/icons/hicolor/scalable/autocomplete/ without known owner

Ooh. That's not part of the icon theme, so is probably worth an upstream bug (and owning in this package)?

> E: invalid-appdata-file /usr/share/appdata/org.gnome.Builder.appdata.xml
> interesting.

Probably related to comment #1.

(In reply to David King from comment #6)
> (In reply to Igor Gnatenko from comment #5)
> > directory /usr/share/icons/hicolor/scalable/autocomplete/ without known owner
> 
> Ooh. That's not part of the icon theme, so is probably worth an upstream bug
> (and owning in this package)?

I owned the directory now, and reuploaded the fixed spec and srpm in place.

http://koji.fedoraproject.org/koji/taskinfo?taskID=8707271

APPROVED.

Please add `ignatenkobrain` and `bochecha` to owners when you will create SCM request.

New Package SCM Request
=======================
Package Name: gnome-builder
Short Description: IDE for writing GNOME-based software
Upstream URL: https://wiki.gnome.org/Apps/Builder
Owners: amigadave bochecha ignatenkobrain
Branches:
InitialCC:

Don't add me as owner, add group::gnome-sig ;)

New Package SCM Request
=======================
Package Name: gnome-builder
Short Description: IDE for writing GNOME-based software
Upstream URL: https://wiki.gnome.org/Apps/Builder
Owners: amigadave group::gnome-sig ignatenkobrain
Branches:
InitialCC:

Git done (by process-git-requests).

Thanks for the reviews!