Skip to content

Fix build for the tapi command#17

Merged
tpoechtrager merged 2 commits intotpoechtrager:masterfrom
midchildan:fix/tapi-command
Dec 19, 2020
Merged

Fix build for the tapi command#17
tpoechtrager merged 2 commits intotpoechtrager:masterfrom
midchildan:fix/tapi-command

Conversation

@midchildan
Copy link
Copy Markdown
Contributor

This PR makes it possible to build the tapi command, which can create .tbd files from existing mach-O binaries.

@tpoechtrager
Copy link
Copy Markdown
Owner

Causes the build to fail on Linux:

-- Targeting X86
-- Targeting XCore
-- Looking for sys/resource.h
-- Looking for sys/resource.h - found
-- Clang version: 8.0.0
-- Performing Test CXX_SUPPORTS_NO_NESTED_ANON_TYPES_FLAG
-- Performing Test CXX_SUPPORTS_NO_NESTED_ANON_TYPES_FLAG - Success
-- Could NOT find Z3 (missing: Z3_LIBRARIES Z3_INCLUDE_DIR) (Required is at least version "4.7.1")
-- TAPI version: 11.0.0
-- Supported Architectures: i386;x86_64;x86_64h;armv4t;armv6;armv5;armv7;armv7s;armv7k;armv6m;armv7m;armv7em;arm64;arm64e
-- Performing Test LINKER_SUPPORTS_NO_INITS
-- Performing Test LINKER_SUPPORTS_NO_INITS - Failed
-- Performing Test LINKER_SUPPORTS_IOSMAC
-- Performing Test LINKER_SUPPORTS_IOSMAC - Failed
-- Performing Test SDK_SUPPORTS_I386
-- Performing Test SDK_SUPPORTS_I386 - Failed
CMake Error at projects/libtapi/tools/tapi/CMakeLists.txt:18 (target_link_libraries):
The keyword signature for target_link_libraries has already been used with
the target "tapi".  All uses of target_link_libraries with a target must be
either all-keyword or all-plain.

The uses of the keyword signature are here:

* cmake/modules/AddLLVM.cmake:787 (target_link_libraries)



-- Failed to find LLVM FileCheck
-- Found Git: /usr/bin/git (found version "2.29.2") 
-- git Version: v0.0.0
-- Version: 0.0.0
-- Performing Test HAVE_CXX_FLAG_STD_CXX11
-- Performing Test HAVE_CXX_FLAG_STD_CXX11 - Success


$ cmake --version
cmake version 3.19.2

CMake suite maintained and supported by Kitware (kitware.com/cmake).

@midchildan
Copy link
Copy Markdown
Contributor Author

I've fixed the build for Linux.

@tpoechtrager tpoechtrager merged commit 664b841 into tpoechtrager:master Dec 19, 2020
@midchildan midchildan deleted the fix/tapi-command branch December 20, 2020 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants