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

Bug#1110968: lintian: fails to process qps_2.10.0-1_amd64.deb, angband-data_4.2.5+dfsg1-3_all.deb, gedit-plugin-smart-spaces_48.1-2_amd64.deb: YAML::XS::Load Error: The problem: mapping keys are not allowed in this context



affects 1094274 lintian
thanks

This issue seems to be caused by #1094274 in the appstream package.

To verify if a package has valid appstream metadata, we run:

    appstreamcli validate-tree --format=yaml --no-net

The resulting YAML is then ingested with YAML::XS::Load() to try to parse it.

The problem is #1094274 causes the YAML to be invalid, since it adds a spurious line at the end. For example, this is what the YAML output looks like for qps_2.10.0-1_amd64:

=========
---
File: org.lxqt.Qps.appdata.xml
Validator: 1.0.6
Issues:
- tag: spdx-license-unknown
  severity: warning
  component: org.lxqt.qps
  line: 5
  hint: GPLv2
  explanation: >-
    The license ID was not found in the SPDX database. Please check that the license ID is written in
    an SPDX-conformant way and is a valid free software license.
- tag: description-first-para-too-short
  severity: info
  component: org.lxqt.qps
  line: 9
  hint: "Process manager written in Qt. Among\n    its features are:"
  explanation: >-
    The first `description/p` paragraph of this component might be too short (< 80 characters). Please
    consider starting with a longer paragraph to improve how the description looks like in software centers
    and to provide more detailed information on this component immediately in the first paragraph.
- tag: screenshot-no-caption
  severity: pedantic
  component: org.lxqt.qps
  line: 24
  explanation: >-
    The screenshot does not have a caption text. Consider adding one.
- tag: releases-info-missing
  severity: pedantic
  component: org.lxqt.qps
  explanation: >-
    This component is missing information about releases. Consider adding a `releases` tag to describe
    releases and their changes.
- tag: content-rating-missing
  severity: info
  component: org.lxqt.qps
  explanation: >-
    This component has no `content_rating` tag to provide age rating information. You can generate the
    tag data online by answering a few questions at https://hughsie.github.io/oars/
- tag: developer-info-missing
  severity: info
  component: org.lxqt.qps
  explanation: >-
    This component contains no `developer` element with information about its author.
- tag: metainfo-filename-cid-mismatch
  severity: warning
  component: org.lxqt.qps
  explanation: >-
    The metainfo filename does not match the component ID.
Passed: no
---
File: qps.desktop
Validator: 1.0.6
Issues:
- tag: desktop-entry-bad-data
  hint: Key file contains key ?Comment[es]? which has a value that cannot be interpreted.
Passed: yes

(appstreamcli:58963): GLib-GObject-CRITICAL **: 04:06:57.444: g_object_unref: assertion 'G_IS_OBJECT (object)' failed
=========

We _could_ fix this issue by wrapping the YAML::XS::Load() in a try/catch statement, but I'd much rather the actual bug be solved instead.

--
  ⢀⣴⠾⠻⢶⣦⠀
  ⣾⠁⢠⠒⠀⣿⡁  Louis-Philippe Véronneau
  ⢿⡄⠘⠷⠚⠋   pollo@debian.org / veronneau.org
  ⠈⠳⣄

Attachment: OpenPGP_0xE1E5457C8BAD4113.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature


Reply to: