[Lldb-commits] [lldb] [lldb][cmake] Remove unnecessary target dependency (PR #150300)

[Chelsea Cassanova via lldb-commits]

https://github.com/chelcassanova created https://github.com/llvm/llvm-project/pull/150300

On the LLDB GreenDragon standalone build, there's currently an error at config time wherein the target libldlb-resource-headers and the individual targets that run the framework fixup script on the header files for the framework (lldb-framework-fixup-header-<header-filename>) both depend on the header file being staged, but neither of which depend on each other. That dependency could be introduced, but I think it would be better if liblldb-resource-headers no longer had a file-based dependency on the staged headers as this is now taken care of by the target lldb-framework-fixup-all-headers.

>From 7c6de98760e412f63e70dc6deef0202efebd9c79 Mon Sep 17 00:00:00 2001
From: Chelsea Cassanova <chelsea_cassanova at apple.com>
Date: Wed, 23 Jul 2025 10:15:50 -0700
Subject: [PATCH] [lldb][cmake] Remove unnecessary target dependency

On the LLDB GreenDragon standalone build, there's currently an error
at config time wherein the target libldlb-resource-headers and the
individual targets that run the framework fixup script on the header
files for the framework (lldb-framework-fixup-header-<header-filename>)
both depend on the header file being staged, but neither of which depend
on each other. That dependency could be introduced, but I think it would
be better if liblldb-resource-headers no longer had a file-based
dependency on the staged headers as this is now taken care of by the
target lldb-framework-fixup-all-headers.
---
 lldb/cmake/modules/LLDBFramework.cmake | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/lldb/cmake/modules/LLDBFramework.cmake b/lldb/cmake/modules/LLDBFramework.cmake
index c6f00ed05cfc2..ef25684c81fc0 100644
--- a/lldb/cmake/modules/LLDBFramework.cmake
+++ b/lldb/cmake/modules/LLDBFramework.cmake
@@ -71,12 +71,8 @@ endif()
 find_program(unifdef_EXECUTABLE unifdef)
 
 # Wrap output in a target, so lldb-framework can depend on it.
-add_custom_target(liblldb-resource-headers DEPENDS lldb-sbapi-dwarf-enums ${lldb_staged_headers})
+add_custom_target(liblldb-resource-headers DEPENDS lldb-sbapi-dwarf-enums)
 set_target_properties(liblldb-resource-headers PROPERTIES FOLDER "LLDB/Resources")
-# We're taking the header files from where they've been staged in the build directory's include folder,
-# so create a dependency on the build step that creates that directory.
-add_dependencies(liblldb-resource-headers liblldb-header-staging)
-add_dependencies(liblldb liblldb-resource-headers)
 
 # Copy vendor-specific headers from clang (without staging).
 if(NOT APPLE_EMBEDDED)