Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 827498 - kde-frameworks/prison: Dmtx and ZXing are optional dependencies
Summary: kde-frameworks/prison: Dmtx and ZXing are optional dependencies
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-26 15:47 UTC by Bug Bugs
Modified: 2021-11-30 08:58 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bug Bugs 2021-11-26 15:47:07 UTC
media-libs/zxing-cpp is relatively big, ~80MB source file to fetch.

Compared to ~80MB of dev-db/mariadb files I had in the same update list. Also this barcode thing is questionable for desktop, even if kde-frameworks/prison itself is required dependency (for kde-plasma/plasma-workspace (?)) it's parts should not be forced to install.

Reproducible: Always
Comment 1 Andreas Sturmlechner gentoo-dev 2021-11-26 17:27:36 UTC
Barcode/qrcode support is what prison does, it would be odd to provide 'less' barcode support for saving a handful of MBs - I don't consider 80 MB anything to write home about. Rather look at the dependencies of media-libs/zxing-cpp:

 * dependency graph for media-libs/zxing-cpp-1.2.0
 `--  media-libs/zxing-cpp-1.2.0  ~amd64 
   `--  dev-util/ninja-1.10.2-r1  (dev-util/ninja) ~amd64 
   `--  dev-util/cmake-3.22.0  (>=dev-util/cmake-3.20.5) ~amd64

Basically, nothing you don't already have installed.

Install size is negligible:

$ equery s zxing-cpp
 * media-libs/zxing-cpp-1.2.0
         Total files : 179
         Total size  : 2.02 MiB

All of this download size comes from tests basically.

Longterm, this library could replace those functions currently provided by libdmtx and qrencode, and then the lack of 'REQUIRED' keyword upstream seems like an omission.

If we were to make this optional, the IUSE would be pdf417 or similar - difficult to communicate. Then there is already a revdep depending on this particular support, so we would make it default enabled on prison.
Comment 2 Bug Bugs 2021-11-26 18:42:17 UTC
I can both agree and disagree with you.I agree about the dependencies, but disagree about resources used by the package. They do not belong to package, game, etc., the are shared. So 80 MB used by a single package and there are dozens of such packages in typical desktop system and depending how frequently they updated it will be noticeable for [disk space|bandwidth|compile time|app mem usage|...]. Well, I don't want to discuss "But it's only XXX, modern systems usually have XXXX ...".

But, there is "qrcode" USE flag for kde-plasma/plasma-meta
"qrcode : Enable support for easy WiFi network sharing using QR codes with kde-plasma/plasma-nm"
which controls kde-frameworks/prison dependency and seems do nothing since kde-frameworks/prison is REQUIRED dependency for kde-plasma/plasma-workspace. So maybe there is a way to attach media-libs/zxing-cpp to "qrcode" USE flag?

If I understand correctly most such plasma or frameworks REQUIRED dependencies are used|required  in "headers-only" mode (e.g. wayland integration).
Comment 3 Andreas Sturmlechner gentoo-dev 2021-11-26 18:53:03 UTC
(In reply to Bug Bugs from comment #2)
> But, there is "qrcode" USE flag for kde-plasma/plasma-meta
> "qrcode : Enable support for easy WiFi network sharing using QR codes with
> kde-plasma/plasma-nm"
The sole reason for it was to enforce kde-frameworks/prison[qml] which is a runtime-only dependency, not the case here. It is already obsolete because kde-plasma/plasma-workspace mandates qml usedep anyway and subsequently will be removed in 5.23.4.

(In reply to Bug Bugs from comment #2)
> So maybe there is a way to attach media-libs/zxing-cpp to "qrcode" USE flag?
That would be incorrect.

(In reply to Bug Bugs from comment #2)
> If I understand correctly most such plasma or frameworks REQUIRED
> dependencies are used|required  in "headers-only" mode (e.g. wayland
> integration).
Not sure where you got that impression.
Comment 4 Andreas Sturmlechner gentoo-dev 2021-11-26 19:03:34 UTC
(In reply to Bug Bugs from comment #2)
> So 80 MB used by a single package and there are dozens of such packages in
> typical desktop system and depending how frequently they updated it will be
> noticeable for [disk space|bandwidth|compile time|app mem usage|...].
We're in 2021, not in 2005, but even then: Last release was in May, so you may divide those 80 MB in bandwidth through 7 months to put it in relation. Not a very high frequency release package.