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)

patch-link-iculibs.diff (251 bytes) - added by RJVB (René Bertin) 5 years ago.

Download all attachments as: .zip

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: newassigned

comment:3 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)

Status: assignedaccepted
Summary: [icu-le-hb] : missing link with ICU librariesicu-le-hb @1.0.3: missing link with ICU libraries

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.

comment:4 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: acceptedclosed

In 4149559ab92b68ea41c1d7f8c8638ad9a0a5ed18/macports-ports (master):

icu-le-hb: Backport upstream changes since 1.0.3

Closes: #59522

Note: See TracTickets for help on using tickets.