Welcome to CRUX bug tracking.

FS#1800 - qt5 missing icu as a dependency

Attached to Project: CRUX
Opened by Alan Mizrahi (alancio) - Thursday, 26 March 2020, 02:08 GMT
Last edited by Danny Rawlins (Romster) - Friday, 10 April 2020, 05:43 GMT
Task Type Bug Report
Category ports → core/opt
Status Closed
Assigned To Danny Rawlins (Romster)
Operating System CRUX
Severity Low
Priority Normal
Reported Version 3.5
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


qt5 missing icu as a dependency, which causes issues with package build order, etc.
This task depends upon

Closed by  Danny Rawlins (Romster)
Friday, 10 April 2020, 05:43 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in commit 20dc08ed42512c18489d5d12c7e386cb19a8788b
Comment by Fredrik Rinnestam (frinnst) - Thursday, 26 March 2020, 17:05 GMT
qt5 builds fine without icu. It's not an explicit dep but I suspect it will try to link against it if available. That could cause problems if icu is somehow broken
Comment by Alan Mizrahi (alancio) - Friday, 27 March 2020, 01:01 GMT
It seems like qt5 ships with its own version of icu, and only links against it when there is no system-wide icu installed. The problem is when qt5 is linked against the system-wide icu, updating the latter will break the former.

Generally speaking, linking against system-wide versions of libraries has the benefit of size reductions (eg: disk, memory), and the possibility of minor version library updates without the need to rebuild dependent applications

Not having it as a dependency means tools like prt-get won't know to update icu before updating qt5, so if you still don't want to add it as a dependency, how about listing it as "Optional"?
Comment by Danny Rawlins (Romster) - Saturday, 28 March 2020, 11:57 GMT
I have purposelessly not listed icu so it is a soft dependency. I am pretty sure if I did list icu as a dependency I would get users asking why.

I will add it as Optional: icu