From 53eb0704115bdd30dca292a8540f9705870f1e93 Mon Sep 17 00:00:00 2001 From: Elias Bachaalany Date: Thu, 12 Mar 2026 11:35:29 -0700 Subject: [PATCH 1/2] fix: remove GIT_SHALLOW for pinned fastmcpp commit hash MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Shallow clones can't resolve a specific SHA — only branches/tags. --- CMakeLists.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index fd53d37..f7d0d20 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -99,7 +99,6 @@ if(IDASQL_WITH_MCP) FetchContent_Declare(fastmcpp GIT_REPOSITORY https://github.com/0xeb/fastmcpp.git GIT_TAG 4837e8e0a27d0011b73643fe02b3553cd3935c34 - GIT_SHALLOW TRUE ) FetchContent_MakeAvailable(fastmcpp) endif() From c0eac083600b73d8775e3b221658bbff812c013f Mon Sep 17 00:00:00 2001 From: Elias Bachaalany Date: Thu, 12 Mar 2026 11:35:34 -0700 Subject: [PATCH 2/2] fix: simplify RPATH to match SDK GNU make convention MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove hardcoded $IDASDK/src/bin from RPATH — it can resolve to stub dylibs instead of real runtime libraries, causing symbol failures when opening databases. Now uses only @executable_path/ (macOS) and $ORIGIN (Linux), matching the SDK's own idalib makefile. The binary deploys to IDABIN via RUNTIME_OUTPUT_DIRECTORY, so the paths resolve correctly. Closes #21 --- src/cli/CMakeLists.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cli/CMakeLists.txt b/src/cli/CMakeLists.txt index 7367b96..83a6386 100644 --- a/src/cli/CMakeLists.txt +++ b/src/cli/CMakeLists.txt @@ -39,12 +39,12 @@ if(MSVC) elseif(APPLE) set_target_properties(idasql_cli PROPERTIES BUILD_WITH_INSTALL_RPATH TRUE - INSTALL_RPATH "@executable_path;$ENV{IDASDK}/src/bin" + INSTALL_RPATH "@executable_path/" ) elseif(UNIX) set_target_properties(idasql_cli PROPERTIES BUILD_WITH_INSTALL_RPATH TRUE - INSTALL_RPATH "$ORIGIN;$ENV{IDASDK}/src/bin" + INSTALL_RPATH "$ORIGIN" ) endif()