plasma_install_package vs plasma_install_bundled_package

[Marco Martin]

At some point we decided we wanted to ship plasmoids in qrc files.
we started to do it, but then between bugs, and people strongly
disagreeing we reverted all of it just before the plasma release
(however, framework was already released, so had to stay)

So i guess the whole plasma_install_bundled_package should be
deprecated now, unless someone really feels is really the way to go...

On Thu, Jul 29, 2021 at 2:29 PM Nicolas Fella <nicolas.fella at gmx.de> wrote:
>
> Hi,
>
> in
> https://invent.kde.org/frameworks/plasma-framework/-/blob/master/KF5PlasmaMacros.cmake#L12
> we recommend using plasma_install_bundled_package over
> plasma_install_package.
>
> However in practice we always use the latter, except for plasma-pa and
> plasma-nm.
>
> I stumbled across this because the bundled version does not seem to
> create a metadata.json from the metadata.desktop like the other macro
> does. That means that the desktop file needs to be converted to json at
> runtime which comes with a significant performance overhead when loading
> a plasmoid.
>
> Now I'm wondering whether we should fix plasma_install_bundled_package
> or port the two users away from it and abandon it.
>
> Thoughts?
>
> Cheers
>
> Nico
>