Make LLVM/Clang include dirs PUBLIC for tools

artemdinaburg · claude · artemdinaburg · commit d3c02dc60d7e · 2025-12-04T15:32:00.000-05:00
The tools (rellic-decomp, rellic-headergen) include Clang headers but weren't getting the include paths since they were PRIVATE to the library. This caused build failures when using system LLVM packages. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/lib/CMakeLists.txt b/lib/CMakeLists.txt
@@ -131,7 +131,6 @@ target_include_directories("${PROJECT_NAME}"
   PUBLIC
     $<BUILD_INTERFACE:${RELLIC_SOURCE_DIR}/include>
     $<INSTALL_INTERFACE:include>
-  PRIVATE
     ${LLVM_INCLUDE_DIRS}
     ${CLANG_INCLUDE_DIRS}
 )

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,6 @@ target_include_directories("${PROJECT_NAME}"`
`131`	`131`	`PUBLIC`
`132`	`132`	`$<BUILD_INTERFACE:${RELLIC_SOURCE_DIR}/include>`
`133`	`133`	`$<INSTALL_INTERFACE:include>`
`134`		`- PRIVATE`
`135`	`134`	`${LLVM_INCLUDE_DIRS}`
`136`	`135`	`${CLANG_INCLUDE_DIRS}`
`137`	`136`	`)`