Bug 759923 - Control whether icu support is built
Summary: Control whether icu support is built
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: qt
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-04 21:38 UTC by Ville Skyttä
Modified: 2012-01-04 01:51 UTC (History)
10 users (show)

Fixed In Version: qt-4.8.0-5.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-04 01:51:42 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ville Skyttä 2011-12-04 21:38:09 UTC
qt's build autodetects icu and enables it if available; at least for reproducible builds enabling it should be better controlled in the specfile.  Enabling it would apparently be a matter of "BuildRequires: libicu-devel" (haven't looked into whether it'd be actually desirable).

Comment 1 Rex Dieter 2011-12-05 15:08:13 UTC
thanks!


%changelog
* Sun Dec 04 2011 Rex Dieter <rdieter> 4.8.0-0.28.rc1
- Control whether icu support is built (#759923)

Comment 2 Ville Skyttä 2011-12-05 16:00:15 UTC
Having looked at the results, I think that if icu support is turned on, an explicit dependency on libicu should also be added.  qt doesn't link to those libs but dynamically opens them so there are no automatic lib dependencies, see code in src/corelib/tools/qlocale_icu.cpp, qt_initIcu().

The explicit libicu dep should be versioned too, as the build time U_ICU_VERSION_SHORT (which is for example "48" in the icu 4.8 series) is used when loading the lib.

Comment 3 Rex Dieter 2011-12-05 16:10:42 UTC
oh, arg, what's the upstream aversion to simply linking it here?  :-/

k, thanks for the extra diligence to notice all the details i've missed.

Comment 4 Ville Skyttä 2011-12-05 18:50:12 UTC
NP, and there's always the option to explicitly disable ICU support if the problems enabling it outweigh the benefits (and I still haven't looked closely enough to be able to tell exactly what the benefits would be, nor do I actually plan to :))

Comment 5 Fedora Update System 2011-12-14 12:10:15 UTC
qt-4.8.0-0.29.rc1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/qt-4.8.0-0.29.rc1.fc16

Comment 6 Ville Skyttä 2011-12-14 22:06:50 UTC
The problem described in comment 2 is not yet taken care of in this build (icu support is built, but there's no icu dependency).  I haven't tested, but I suppose it could cause regressions.

Comment 7 Rex Dieter 2011-12-16 14:06:52 UTC
OK, after a cursory evaluation, at this time:

* we feel the code paths with and without libicu being present are relatively safe

* hard-coding a new ~19mb runtime dependency would be undesirable (especially for our admittedly english-only and space-constrained livecd)

* given the benefits of having libicu support at runtime, however, we're seriously considering adding it as a default item in @kde-desktop comps group

Comment 8 Vedran Miletić 2011-12-19 11:01:15 UTC
Are you sure there are no crashes if libicu is missing?

Comment 9 Rex Dieter 2011-12-19 17:11:11 UTC
Confirmed no crashes, but ... 

do get some potentially annoying spammage in ~/.xsession-errors:

Unable to load library icui18n "Cannot load library icui18n: (libicui18n.so.46: cannot open shared object file: No such file or directory)"

on launch of any qt (or kde) application.  :(

Comment 10 Rex Dieter 2011-12-19 17:24:50 UTC
So, I guess we've got some options: 
1. do nothing
2. remove/silence the qWarning call in qlocale_icu.cpp  (assuming building qt with -DQT_NO_DEBUG_OUTPUT and QT_NO_WARNING_OUTPUT is overkill).
3. add the dep to qt after all
4. some better idea

Comment 11 Ville Skyttä 2011-12-19 19:24:27 UTC
Better or worse, dunno, but one more option nevertheless:

5. explicitly build without icu

Comment 12 Fedora Update System 2011-12-20 23:41:04 UTC
qt-4.8.0-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/qt-4.8.0-3.fc16

Comment 13 Fedora Update System 2011-12-22 22:45:49 UTC
Package qt-4.8.0-3.fc16:
* should fix your issue,
* was pushed to the Fedora 16 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing qt-4.8.0-3.fc16'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2011-17326/qt-4.8.0-3.fc16
then log in and leave karma (feedback).

Comment 14 Fedora Update System 2011-12-22 22:47:24 UTC
qt-4.8.0-0.29.rc1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2012-01-04 01:51:42 UTC
qt-4.8.0-5.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.