Data build for MacOSX broken with --enable-rpath

Description

in fixing I broke `config --enable-rpath`

{{…
pkgdata: clang -dynamiclib -dynamic -O2 -std=c11 -Wall -pedantic -Wshadow -Wpointer-arith -Wmissing-prototypes -Wwrite-strings -Qunused-arguments -Wno-parentheses-equality -Wl,-rpath,/usr/local/lib -o ../lib/libicudata.64.2.dylib ./out/tmp/icudt64l_dat.o -Wl,-compatibility_version -Wl,64 -Wl,-current_version -Wl,64.2 -install_name /usr/local/lib/ libicudata.64.dylib
clang: error: no such file or directory: 'libicudata.64.dylib'}}

so in this case (rpath) there should NOT be a trailing space at the end of LD_SONAME:

`… -install_name /usr/local/lib/`

whereas in the non-rpath version, there SHOULD be a space:

`… -install_name `

Activity

Show:
Steven R. Loomis
April 17, 2019, 4:12 PM
Edited

There’s also a workaround. After you have configured with --enable-rpath on OSX, but before the first make invocation, run:

 

 

make PKGDATA_TRAILING_SPACE=

 

(No, there’s no trailing space in that command. That’s the workaround.)

 

If you already see the failure, you can run make cleanin the data subdirectory and then the above make.

Steven R. Loomis
April 17, 2019, 2:05 AM

Ok I think I found a fix. The rpath version in the mh- file entry for LD_SONAME ends with a directory and a / which is to have the library name appended. This entry should not have the trailing space hack I put in for ICU-20526.

Fixed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Steven R. Loomis

Reporter

Steven R. Loomis

Components

Reviewer

Steven R. Loomis

Priority

blocks-progress

Fix versions