set lzo compression by default in kde-neon extension

[jriddell]

• [x ] Have you followed the guidelines for contributing?
• [x ] Have you signed the CLA?
• [x ] Have you successfully run ./runtests.sh static?
• [x ] Have you successfully run ./runtests.sh tests/unit?

enable lzo compression in kde-neon extension by default

from chatting with ppd and post at https://forum.snapcraft.io/t/lzo-for-platform-snaps/27501

---

[ppd]

Is this something we still want to do? Electron-builder switch to LZO by default, too.

[sergiusens]

sorry @ppd for the delays, let's do this

[sergiusens]

want me to finalize the PR?

[ppd]

want me to finalize the PR?

@jriddell ping

[sergiusens]

@igorljubuncic maybe you want to quickly profile the performance gains of switching to lzo for kde apps (the heavy and light ones)?

[ghost]

Yes, this would be a useful test!

[ppd]

@igorljubuncic The same idea would apply well to Gnome-based snaps. They could also be compressed as LZO by default. Furthermore, what about compressing the platform snaps as LZO? The kde platform snaps switched to LZO and saw read performance gains, which propagated through to the app snaps.

This could be a rather cheap "make desktop snaps fast" initiative.

[ghost]

@ppd yes, but the best thing is to do it systematically, step by step. I spoke with @jriddell and he will build several kde snaps as well as the framework with/without lzo, and then we can run tests and examine the performance. Then, this can be further extended to additional snaps.

[jriddell]

kblocks
https://snapcraft.io/kblocks
rev 113 in beta channel is xz (4.8MB)
rev 114 in edge channel is lzo (6.2MB)

gcompris
https://snapcraft.io/gcompris
rev 35 in beta channel is xz (152MB)
rev 36 in edge channel is lzo (186MB)

[jriddell]

kde-frameworks-5-qt-5-15-3-core20
rev 12 in beta is xz
rev 13 in edge is lzo

[sergiusens]

If you don't mind, I am going to prep this PR for release

[codecov-commenter]

Codecov Report

Merging #3595 (10c69fb) into main (183fb9d) will not change coverage.
The diff coverage is n/a.

❗ Current head 10c69fb differs from pull request most recent head 4cd4173. Consider uploading reports for the commit 4cd4173 to get more accurate results

@@           Coverage Diff           @@
##             main    #3595   +/-   ##
=======================================
  Coverage   91.25%   91.25%           
=======================================
  Files         278      278           
  Lines       19396    19396           
=======================================
  Hits        17699    17699           
  Misses       1697     1697           

Impacted Files	Coverage Δ
...ft/internal/project_loader/_extensions/kde_neon.py	100.00% <ø> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more