Make AppStream generation opt-in
AbandonedPublic

Authored by apol on Mar 18 2019, 10:52 AM.

Details

Reviewers
ngraham
Group Reviewers
Plasma
Summary

After discussing the feature over it became apparent that developers will need to be pro-active at generating the metadata whenever it's needed and that it's not clear that having everything is overall useful.
This change will make the kpackage_install*_package() calls not install an appdata.xml file by default.

CCBUG: 403118

Test Plan

Plasma Workspace doesn't install appdata anymore

Diff Detail

Repository
R290 KPackage
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 9777
Build 9795: arc lint + arc unit
apol created this revision.Mar 18 2019, 10:52 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptMar 18 2019, 10:52 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
apol requested review of this revision.Mar 18 2019, 10:52 AM

Conceptual +1; giving everything autogenerated yet sparse AppStream data doesn't provide much value and causes various problems with how app store apps (e.g. Discover) present our software. See https://bugs.kde.org/show_bug.cgi?id=403120. AppStream data should ideally be handmade by the developer because the whole point (well, one of them) is that it's for marketing and branding purposes.

ngraham edited the summary of this revision. (Show Details)Mar 18 2019, 2:36 PM
sitter added a subscriber: sitter.Mar 27 2019, 1:17 PM

LGTM technically. Also looks crazy scary invasive though from a packaging POV :O

What if a third party user of kpackage actively relied on the appstream data being auto-generated? Wouldn't this change in behavior essentially break backwards compatibility? Perhaps it'd be less scary if ECM's KDECmakeSettings set some var to make KDE software default-opt-out of autogeneration (and then revisit the default behavior for the macro for KF6)?

ngraham accepted this revision.Mar 27 2019, 2:15 PM
This revision is now accepted and ready to land.Mar 27 2019, 2:15 PM

LGTM technically. Also looks crazy scary invasive though from a packaging POV :O

What if a third party user of kpackage actively relied on the appstream data being auto-generated?

Then they'd basically be doing it wrong since you're supposed to actually put some effort into creating the appdata file yourself. :)

Perhaps it'd be less scary if ECM's KDECmakeSettings set some var to make KDE software default-opt-out of autogeneration (and then revisit the default behavior for the macro for KF6)?

No strong objection.

Ping! Can we land this or a variant of it?

apol abandoned this revision.Apr 26 2019, 2:47 PM