[MacPorts] #59522: [icu-le-hb] : missing link with ICU libraries

[MacPorts]

#59522: [icu-le-hb] : missing link with ICU libraries
------------------------+----------------------
  Reporter:  RJVB       |      Owner:  (none)
      Type:  defect     |     Status:  new
  Priority:  Normal     |  Milestone:
 Component:  ports      |    Version:
Resolution:             |   Keywords:  haspatch
      Port:  icu-le-hb  |
------------------------+----------------------
Description changed by RJVB:

Old description:

> 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).

New description:

 (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).

--

-- 
Ticket URL: <https://trac.macports.org/ticket/59522#comment:1>
MacPorts <https://www.macports.org/>
Ports system for macOS