From: Igor Munkin via Tarantool-patches <tarantool-patches@dev.tarantool.org> To: Sergey Ostanevich <sergos@tarantool.org>, Sergey Kaplun <skaplun@tarantool.org> Cc: tarantool-patches@dev.tarantool.org Subject: [Tarantool-patches] [PATCH luajit 1/3] build: pass sysroot to MacOS SDK Date: Tue, 11 May 2021 01:09:05 +0300 [thread overview] Message-ID: <4dd97d6fbfb470d41e9ef5ad12492a0def664b2b.1620678384.git.imun@tarantool.org> (raw) In-Reply-To: <cover.1620678384.git.imun@tarantool.org> There were issues with configuring LuaJIT on Apple machines, since <LuaJITTestArch> CMake auxiliary routine fails to locate system headers (e.g. assert.h in case when LUA_USE_ASSERT is enabled). As a result platform detection fails and LuaJIT configuration ends with the fatal error. This patch adds the necessary flags to help the routine to find the required system headers. Relates to tarantool/tarantool#5629 Needed for tarantool/tarantool#5983 Follows up tarantool/tarantool#4862 Signed-off-by: Igor Munkin <imun@tarantool.org> --- CMakeLists.txt | 9 +++++++++ cmake/LuaJITUtils.cmake | 7 ++++++- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 5348e043..110a989f 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -114,6 +114,15 @@ if(LUAJIT_ENABLE_WARNINGS) ) endif() +# Set sysroot settings on OSX to find SDK with the system headers. +# XXX: Obviously, there is no need in this setup if everything is +# already set via CMAKE_C_FLAGS or in parent project build system. +if(CMAKE_OSX_SYSROOT AND CMAKE_C_SYSROOT_FLAG AND + NOT "${CMAKE_C_FLAGS}" MATCHES "${CMAKE_C_SYSROOT_FLAG} ${CMAKE_OSX_SYSROOT}" +) + AppendFlags(CMAKE_C_FLAGS "${CMAKE_C_SYSROOT_FLAG} ${CMAKE_OSX_SYSROOT}") +endif() + # Auxiliary flags for main targets (libraries, binaries). AppendFlags(TARGET_C_FLAGS -D_FILE_OFFSET_BITS=64 diff --git a/cmake/LuaJITUtils.cmake b/cmake/LuaJITUtils.cmake index d9f8b12a..8ff26a6a 100644 --- a/cmake/LuaJITUtils.cmake +++ b/cmake/LuaJITUtils.cmake @@ -1,8 +1,13 @@ function(LuaJITTestArch outvar strflags) + # XXX: Compiler flags are also required in this routine. It can + # use e.g. external headers, which location is specified either + # implicitly (within CMake machinery) or explicitly (manually by + # configuration options). + set(TESTARCH_C_FLAGS "${CMAKE_C_FLAGS} ${strflags}") # XXX: <execute_process> simply splits the COMMAND argument by # spaces with no further parsing. At the same time GCC is bad in # argument handling, so let's help it a bit. - separate_arguments(TESTARCH_C_FLAGS UNIX_COMMAND ${strflags}) + separate_arguments(TESTARCH_C_FLAGS UNIX_COMMAND ${TESTARCH_C_FLAGS}) # TODO: It would be nice to drop a few words, why do we use this # approach instead of CMAKE_HOST_SYSTEM_PROCESSOR variable. execute_process( -- 2.25.0
next prev parent reply other threads:[~2021-05-10 22:30 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-10 22:09 [Tarantool-patches] [PATCH luajit 0/3] Basic fixes for LuaJIT on ARM64 Macs Igor Munkin via Tarantool-patches 2021-05-10 22:09 ` Igor Munkin via Tarantool-patches [this message] 2021-05-11 9:49 ` [Tarantool-patches] [PATCH luajit 1/3] build: pass sysroot to MacOS SDK Sergey Kaplun via Tarantool-patches 2021-05-12 21:55 ` Igor Munkin via Tarantool-patches 2021-05-14 16:07 ` Sergey Kaplun via Tarantool-patches 2021-05-17 17:21 ` Igor Munkin via Tarantool-patches 2021-05-18 5:50 ` Sergey Kaplun via Tarantool-patches 2021-05-18 18:47 ` Igor Munkin via Tarantool-patches 2021-05-19 11:38 ` Igor Munkin via Tarantool-patches 2021-05-19 12:40 ` Sergey Ostanevich via Tarantool-patches 2021-05-19 13:23 ` Igor Munkin via Tarantool-patches 2021-05-19 16:06 ` Sergey Kaplun via Tarantool-patches 2021-05-10 22:09 ` [Tarantool-patches] [PATCH luajit 2/3] OSX/iOS: Handle iOS simulator and ARM64 Macs Igor Munkin via Tarantool-patches 2021-05-11 11:02 ` Sergey Kaplun via Tarantool-patches 2021-05-11 11:03 ` Igor Munkin via Tarantool-patches 2021-05-14 11:36 ` Sergey Ostanevich via Tarantool-patches 2021-05-14 11:27 ` Igor Munkin via Tarantool-patches 2021-05-10 22:09 ` [Tarantool-patches] [PATCH luajit 3/3] FFI/ARM64/OSX: Fix vararg call handling Igor Munkin via Tarantool-patches 2021-05-11 11:07 ` Sergey Kaplun via Tarantool-patches 2021-05-11 11:31 ` Igor Munkin via Tarantool-patches 2021-05-12 16:11 ` Sergey Ostanevich via Tarantool-patches 2021-05-12 21:59 ` Igor Munkin via Tarantool-patches 2021-05-13 9:50 ` Sergey Ostanevich via Tarantool-patches 2021-05-13 10:44 ` Igor Munkin via Tarantool-patches 2021-05-14 10:10 ` Sergey Ostanevich via Tarantool-patches 2021-05-14 10:31 ` Igor Munkin via Tarantool-patches 2021-05-19 15:38 ` [Tarantool-patches] [PATCH luajit 0/3] Basic fixes for LuaJIT on ARM64 Macs Igor Munkin via Tarantool-patches
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4dd97d6fbfb470d41e9ef5ad12492a0def664b2b.1620678384.git.imun@tarantool.org \ --to=tarantool-patches@dev.tarantool.org \ --cc=imun@tarantool.org \ --cc=sergos@tarantool.org \ --cc=skaplun@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH luajit 1/3] build: pass sysroot to MacOS SDK' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox