diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2020-02-14 15:23:15 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2020-02-14 17:29:45 +0100 |
commit | b7fd89100d8653dc73955780358fe31d38b68ebf (patch) | |
tree | 61130b26cab8aa97eceafd283e0f850cdeeb6010 /desktop/Executable_soffice_bin.mk | |
parent | 391613785ae6fbb735cf7a86ea2f6a93161a8769 (diff) |
tdf#122218: Baseline Xcode 9.3 ld presumably doesn't support -platform_version
...according to <https://github.com/llvm/llvm-project/commit/
25ce33a6e4f3b13732c0f851e68390dc2acb9123> "[driver][darwin] Pass
-platform_version flag to the linker instead of the -<platform>_version_min
flag": "In Xcode 11, ld added a new flag called -platform_version [...] This
patch adopts the new -platform_version flag in Clang, and starts using it by
default, unless a linker version < 520 is passed to the driver."
So detect new HAVE_MACOS_LD_PLATFORMVERSION and adapt
645fe53be0dc36535dba0ed684e21ca4cda80d70 "tdf#122218: Hack to avoid blurry text
with macOS SDK 10.15" accordingly.
(This also changes the passed -platform_verion sdk value from 0.0 to 0.0.0, for
cosmetic consistency with the default Clang behavior cited above. Also, after
f67e5ef9a5c71f3b35b1c67eb72794e44cc15410 "Drop broken filter-out of
-bind_at_load for Executable_soffice_bin on macOS" got included in the meantime,
the surrounding ifeq in desktop/Executable_soffice_bin.mk can be combined now.)
Change-Id: Ie1ddf2d618e2f1232c6b4e17ce17665851f3bd38
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88717
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'desktop/Executable_soffice_bin.mk')
-rw-r--r-- | desktop/Executable_soffice_bin.mk | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/desktop/Executable_soffice_bin.mk b/desktop/Executable_soffice_bin.mk index f9be897ffa1f..0d9646c78563 100644 --- a/desktop/Executable_soffice_bin.mk +++ b/desktop/Executable_soffice_bin.mk @@ -23,15 +23,17 @@ $(eval $(call gb_Executable_add_cobjects,soffice_bin,\ desktop/source/app/main \ )) -ifeq ($(OS),MACOSX) - +ifeq ($(OS)-$(HAVE_MACOS_LD_PLATFORMVERSION),MACOSX-TRUE) # At least when building against SDK 10.15, changing the LC_VERSION_MIN_MACOSX load command's sdk -# value from 10.15 to "n/a" (i.e., 0.0) is necessary to avoid blurry text in the LO UI: +# value from 10.15 to "n/a" (i.e., 0.0.0) is necessary to avoid blurry text in the LO UI (see +# <https://github.com/llvm/llvm-project/commit/25ce33a6e4f3b13732c0f851e68390dc2acb9123> +# "[driver][darwin] Pass -platform_version flag to the linker instead of the +# -<platform>_version_min flag", clang/test/Driver/darwin-ld-platform-version-macos.c in particular, +# for the -platform_version that Clang passes by default to new-enough ld): $(eval $(call gb_Executable_add_ldflags,soffice_bin, \ -Xlinker -platform_version -Xlinker macos -Xlinker $(MAC_OS_X_VERSION_MIN_REQUIRED_DOTS) \ - -Xlinker 0.0 \ + -Xlinker 0.0.0 \ )) - endif ifeq ($(OS),WNT) |