Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installing a package with invalid metadata breaks appstream system-wide. #211

Closed
antonio-rojas opened this issue Dec 14, 2018 · 2 comments
Closed

Comments

@antonio-rojas
Copy link
Contributor

After updating to 0.12.3 I noticed that 'appstreamcli search' no longer worked, it only returned "Some components are invalid. See debug output for details". The debug output showed the line "WARNING: Ignored component 'org.kde.choqok.desktop': The component is invalid.". And indeed after uninstalling choqok 'appstreamcli search' started working again without touching the appstream index.
I find it surprising that simply installing a package with invalid metadata can break appstream system-wide, when the appstream index itself is correct. It should simply ignore the invalid file (as the debug log says, by the way)

I've bisected it to 5161c4b

@antonio-rojas antonio-rojas changed the title Installing a packages with invalid metadata breaks appstream system-wide. Installing a package with invalid metadata breaks appstream system-wide. Dec 14, 2018
@ximion ximion closed this as completed in f199a6d Dec 16, 2018
@ximion
Copy link
Owner

ximion commented Dec 16, 2018

I originally wanted to resolve this by adjusting appstreamcli to not fail hard on cache loading errors, but eventually opted for still failing hard, but only if there was a large amount of components being recognized as invalid (at least 10% of the total amount of components).
This may get some improvements in future to be less strict about data in the pool, but still complain on issues so they aren't silently ignored.

@ximion
Copy link
Owner

ximion commented Dec 16, 2018

Looks like Choqok need a summary:

E - org.kde.choqok.appdata.xml:org.kde.choqok.desktop
    The component is missing a summary (<summary/> tag).

Validation failed: errors: 1, pedantic: 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants