diff options
author | Jonas Devlieghere <jonas@devlieghere.com> | 2023-01-31 15:33:46 -0800 |
---|---|---|
committer | Jonas Devlieghere <jonas@devlieghere.com> | 2023-01-31 15:34:36 -0800 |
commit | 9f8fd57cb6638f8137d82b1ceb02845626816b4f (patch) | |
tree | eb08c6a2d9cc4a1e6cbf1ad583e38a8bb6103d40 /lldb/cmake | |
parent | 3fe3f9c51cbac91af741e53e96c89f633bd2a6ad (diff) | |
download | llvm-9f8fd57cb6638f8137d82b1ceb02845626816b4f.tar.gz |
[lldb] Use ${CMAKE_COMMAND} -E remove instead of remove_directory
We no longer need to remove a directory downstream and also contrary to
my previous observations, remove_directory isn't sufficient to remove a
regular file.
Differential revision: https://reviews.llvm.org/D143024
Diffstat (limited to 'lldb/cmake')
-rw-r--r-- | lldb/cmake/modules/AddLLDB.cmake | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lldb/cmake/modules/AddLLDB.cmake b/lldb/cmake/modules/AddLLDB.cmake index a5be4afb40fb..251c2957e2d1 100644 --- a/lldb/cmake/modules/AddLLDB.cmake +++ b/lldb/cmake/modules/AddLLDB.cmake @@ -246,10 +246,8 @@ function(lldb_add_to_buildtree_lldb_framework name subdir) # Create a custom target to remove the copy again from LLDB.framework in the # build tree. - # Intentionally use remove_directory because the target can be a either a - # file or directory and using remove_directory is harmless for files. add_custom_target(${name}-cleanup - COMMAND ${CMAKE_COMMAND} -E remove_directory ${copy_dest} + COMMAND ${CMAKE_COMMAND} -E remove ${copy_dest} COMMENT "Removing ${name} from LLDB.framework") add_dependencies(lldb-framework-cleanup ${name}-cleanup) |