Added Support for building on Mac OS with Clang #88
Conversation
fix: Can be compiled on Mac OS with the default system compiler Why: Clang doesn't know the soname linker flag instead the install_name flag is used How: Added LINKER_SONAME_OPTION option to configure script that can be set manually and will be set if the CC env is clang Tags: Clang Support, Mac OS build support, Signed-off-by: SZanko <szanko@protonmail.com>
| @@ -113,10 +114,12 @@ command -v ${CC} 2>/dev/null 1>&2 || { | |||
| echo "${CC} not found: trying clang" 1>&2 | |||
| echo "${CC} not found: trying clang" 1>&3 | |||
| CC=clang | |||
| LINKER_SONAME_OPTION="-install_name" | |||
There was a problem hiding this comment.
This shouldn't be conditional on the type of the C compiler (and the executable name is not a reliable way to detect the type anyways), but instead the type of the linker.
Only macOS' cctools will use -install_name, but clang can also be used with GNU binutils or lld.
|
I think we can use the shared-library OS X format instead of the UNIX-like format. This requires replacing the |
|
Looks like there's some overlap in our PRs -> #108 Any update on getting one of these through? |
|
This has been fixed in commit 0fb3e8f, which incorporates some modifications in oconfigure specific to this issue. Thank you again everybody, and I'm sorry for taking so long! (I'll tag out a release in the next day or so that incorporates this and other pull requests.) |
fix: Can be compiled on Mac OS with the default system compiler(clang)
Why: Clang doesn't know the soname linker flag instead the install_name
flag is used
How: Added LINKER_SONAME_OPTION option to configure script that can be
set manually and will be set if the CC env is clang
Tags: Clang Support, Mac OS build support,
Signed-off-by: SZanko szanko@protonmail.com