Opened 5 years ago
Closed 5 years ago
#59522 closed defect (fixed)
icu-le-hb @1.0.3: missing link with ICU libraries
Reported by: | RJVB (René Bertin) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | |
Port: | icu-le-hb |
Description (last modified by RJVB (René Bertin))
(As discussed on the devel ML)
For some curious reason the icu-le-hb buildsystem fails to link the library with the ICU libraries. This can go undetected as long as those dependencies are loaded first in applications that depend on libicu-le-hb (and thus also as long as you do not load the library in isolation in a thin wrapper around the dlopen()
call).
Apparently that order did change for me, which led to an error from the dynamic linker/loader (dyld.so) about an undefined symbol: _ZTIN6icu_657UObjectE
(the loader bails at the 1st error so we cannot know how many other symbols are undefined too).
The attached patch adds the ICU_LIBS
libraries obtained from pkg-config to the libicu-le-hb link dependencies, which fixes the issue for me (and should be a transparent change otherwise).
Attachments (1)
Change History (5)
Changed 5 years ago by RJVB (René Bertin)
Attachment: | patch-link-iculibs.diff added |
---|
comment:1 Changed 5 years ago by RJVB (René Bertin)
Description: | modified (diff) |
---|
comment:2 Changed 5 years ago by mf2k (Frank Schima)
Cc: | ryandesign removed |
---|---|
Owner: | set to ryandesign |
Status: | new → assigned |
comment:3 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Status: | assigned → accepted |
---|---|
Summary: | [icu-le-hb] : missing link with ICU libraries → icu-le-hb @1.0.3: missing link with ICU libraries |
comment:4 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
Thanks. The same change was already made upstream but there has not been a new release since then. I'll add it to the port.