From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 8A65015800F for ; Wed, 8 Feb 2023 19:12:47 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id CC32FE05C1; Wed, 8 Feb 2023 19:12:46 +0000 (UTC) Received: from smtp.gentoo.org (mail.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id ADFF7E05C1 for ; Wed, 8 Feb 2023 19:12:46 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id DB649335D5D for ; Wed, 8 Feb 2023 19:12:45 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id EC6567D0 for ; Wed, 8 Feb 2023 19:12:43 +0000 (UTC) From: "Sam James" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Sam James" Message-ID: <1675883555.726e8ac1afece6692ca623b7e68b60fd7b7dfc7f.sam@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: app-editors/neovim/, app-editors/neovim/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: app-editors/neovim/files/neovim-9999-cmake-darwin.patch app-editors/neovim/files/neovim-9999-cmake_lua_version.patch app-editors/neovim/neovim-9999.ebuild X-VCS-Directories: app-editors/neovim/files/ app-editors/neovim/ X-VCS-Committer: sam X-VCS-Committer-Name: Sam James X-VCS-Revision: 726e8ac1afece6692ca623b7e68b60fd7b7dfc7f X-VCS-Branch: master Date: Wed, 8 Feb 2023 19:12:43 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 4056b439-e69d-414f-aa3c-8f596d968f6c X-Archives-Hash: e7f84964007abca6d3f23cc7be3c800f commit: 726e8ac1afece6692ca623b7e68b60fd7b7dfc7f Author: Sam James gentoo org> AuthorDate: Wed Feb 8 19:12:14 2023 +0000 Commit: Sam James gentoo org> CommitDate: Wed Feb 8 19:12:35 2023 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=726e8ac1 Revert "app-editors/neovim: fixes (see commit details)" This reverts commit fca327914eac530d8b37251ba459ae8a454de0e1. Breaks applying patches to 0.8.2/0.8.3. Signed-off-by: Sam James gentoo.org> .../neovim/files/neovim-9999-cmake-darwin.patch | 62 ++++++++++++++++++++-- .../files/neovim-9999-cmake_lua_version.patch | 12 ++--- app-editors/neovim/neovim-9999.ebuild | 9 ++-- 3 files changed, 69 insertions(+), 14 deletions(-) diff --git a/app-editors/neovim/files/neovim-9999-cmake-darwin.patch b/app-editors/neovim/files/neovim-9999-cmake-darwin.patch index b2920451743d..ad47a46e1a7a 100644 --- a/app-editors/neovim/files/neovim-9999-cmake-darwin.patch +++ b/app-editors/neovim/files/neovim-9999-cmake-darwin.patch @@ -1,13 +1,65 @@ +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -80,33 +80,6 @@ set(ENV{PKG_CONFIG_PATH} "$ENV{PKG_CONFIG_PATH}:${DEPS_PREFIX}/lib/pkgconfig") + # used for check_c_compiler_flag + include(CheckCCompilerFlag) + +-if(CMAKE_SYSTEM_NAME STREQUAL "Darwin") +- # CMake tries to treat /sw and /opt/local as extension of the system path, but +- # that doesn't really work out very well. Once you have a dependency that +- # resides there and have to add it as an include directory, then any other +- # dependency that could be satisfied from there must be--otherwise you can end +- # up with conflicting versions. So, let's make them more of a priority having +- # them be included as one of the first places to look for dependencies. +- list(APPEND CMAKE_PREFIX_PATH /sw /opt/local) +- +- # If the macOS deployment target is not set manually (via $MACOSX_DEPLOYMENT_TARGET), +- # fall back to local system version. Needs to be done both here and in cmake.deps. +- if(NOT CMAKE_OSX_DEPLOYMENT_TARGET) +- execute_process(COMMAND sw_vers -productVersion +- OUTPUT_VARIABLE MACOS_VERSION +- OUTPUT_STRIP_TRAILING_WHITESPACE) +- set(CMAKE_OSX_DEPLOYMENT_TARGET "${MACOS_VERSION}") +- endif() +- message("Using deployment target ${CMAKE_OSX_DEPLOYMENT_TARGET}") +- +- # Work around some old, broken detection by CMake for knowing when to use the +- # isystem flag. Apple's compilers have supported this for quite some time +- # now. +- if(CMAKE_C_COMPILER_ID MATCHES "GNU") +- set(CMAKE_INCLUDE_SYSTEM_FLAG_C "-isystem ") +- endif() +-endif() +- + if(WIN32 OR CMAKE_SYSTEM_NAME STREQUAL "Darwin") + # Ignore case when comparing filenames on Windows and Mac. + set(CASE_INSENSITIVE_FILENAME TRUE) +--- a/cmake.deps/CMakeLists.txt ++++ b/cmake.deps/CMakeLists.txt +@@ -112,13 +112,6 @@ if(CMAKE_CXX_COMPILER) + set(DEPS_CXX_COMPILER "${CMAKE_CXX_COMPILER}") + endif() + +-if(CMAKE_OSX_SYSROOT) +- set(DEPS_C_COMPILER "${DEPS_C_COMPILER} -isysroot${CMAKE_OSX_SYSROOT}") +- if(DEPS_CXX_COMPILER) +- set(DEPS_CXX_COMPILER "${DEPS_CXX_COMPILER} -isysroot${CMAKE_OSX_SYSROOT}") +- endif() +-endif() +- + if(CMAKE_OSX_ARCHITECTURES) + string(REPLACE ";" "|" CMAKE_OSX_ARCHITECTURES_ALT_SEP "${CMAKE_OSX_ARCHITECTURES}") + # The LuaJIT build does not like being passed multiple `-arch` flags --- a/src/nvim/CMakeLists.txt +++ b/src/nvim/CMakeLists.txt -@@ -440,10 +440,6 @@ get_directory_property(gen_includes INCLUDE_DIRECTORIES) - foreach(gen_include ${prop}) +@@ -181,10 +181,6 @@ get_directory_property(gen_includes INCLUDE_DIRECTORIES) + foreach(gen_include ${gen_includes} ${LUA_PREFERRED_INCLUDE_DIRS}) list(APPEND gen_cflags "-I${gen_include}") endforeach() -if(CMAKE_SYSTEM_NAME STREQUAL "Darwin" AND CMAKE_OSX_SYSROOT) - list(APPEND gen_cflags "-isysroot") - list(APPEND gen_cflags "${CMAKE_OSX_SYSROOT}") -endif() - set(gen_cflags ${gen_cflags} -O2) - - set(NVIM_VERSION_GIT_H ${PROJECT_BINARY_DIR}/cmake.config/auto/versiondef_git.h) + string(TOUPPER "${CMAKE_BUILD_TYPE}" build_type) + separate_arguments(C_FLAGS_ARRAY UNIX_COMMAND ${CMAKE_C_FLAGS}) + separate_arguments(C_FLAGS_${build_type}_ARRAY UNIX_COMMAND ${CMAKE_C_FLAGS_${build_type}}) diff --git a/app-editors/neovim/files/neovim-9999-cmake_lua_version.patch b/app-editors/neovim/files/neovim-9999-cmake_lua_version.patch index b14b3b992349..87a898032493 100644 --- a/app-editors/neovim/files/neovim-9999-cmake_lua_version.patch +++ b/app-editors/neovim/files/neovim-9999-cmake_lua_version.patch @@ -1,11 +1,11 @@ ---- a/src/nvim/CMakeLists.txt -+++ b/src/nvim/CMakeLists.txt +--- a/CMakeLists.txt ++++ b/CMakeLists.txt @@ -464,7 +464,7 @@ endif() - # The unit test lib requires LuaJIT; it will be skipped if LuaJIT is missing. option(PREFER_LUA "Prefer Lua over LuaJIT in the nvim executable." OFF) + if(PREFER_LUA) - find_package(Lua 5.1 EXACT REQUIRED) + find_package(Lua ${PREFER_LUA} EXACT REQUIRED) - target_include_directories(main_lib SYSTEM BEFORE INTERFACE ${LUA_INCLUDE_DIR}) - target_link_libraries(main_lib INTERFACE ${LUA_LIBRARIES}) - # Passive (not REQUIRED): if LUAJIT_FOUND is not set, fixtures for unittests is skipped. + set(LUA_PREFERRED_INCLUDE_DIRS ${LUA_INCLUDE_DIR}) + set(LUA_PREFERRED_LIBRARIES ${LUA_LIBRARIES}) + # Passive (not REQUIRED): if LUAJIT_FOUND is not set, nvim-test is skipped. diff --git a/app-editors/neovim/neovim-9999.ebuild b/app-editors/neovim/neovim-9999.ebuild index c35d1e3b8870..6083e311ff39 100644 --- a/app-editors/neovim/neovim-9999.ebuild +++ b/app-editors/neovim/neovim-9999.ebuild @@ -21,7 +21,7 @@ fi LICENSE="Apache-2.0 vim" SLOT="0" -IUSE="+lto +nvimpager test" +IUSE="+lto +nvimpager test +tui" REQUIRED_USE="${LUA_REQUIRED_USE}" # Upstream say the test library needs LuaJIT @@ -53,8 +53,10 @@ DEPEND="${LUA_DEPS} >=dev-libs/libvterm-0.3 >=dev-libs/msgpack-3.0.0:= >=dev-libs/tree-sitter-0.20.2:= - >=dev-libs/libtermkey-0.22 - >=dev-libs/unibilium-2.0.0:0= + tui? ( + >=dev-libs/libtermkey-0.22 + >=dev-libs/unibilium-2.0.0:0= + ) " RDEPEND=" ${DEPEND} @@ -96,6 +98,7 @@ src_configure() { # TODO: Investigate USE_BUNDLED, doesn't seem to be needed right now local mycmakeargs=( -DENABLE_LTO=$(usex lto) + -DFEAT_TUI=$(usex tui) -DPREFER_LUA=$(usex lua_single_target_luajit no "$(lua_get_version)") -DLUA_PRG="${ELUA}" -DMIN_LOG_LEVEL=3