[clang-tools-extra] [clang-doc] fix bug introduced by asset test (PR #97540)

Paul Kirth via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 3 10:01:07 PDT 2024


================
@@ -25,7 +25,11 @@ set(assets
 )
 
 set(asset_dir "${CMAKE_CURRENT_SOURCE_DIR}/../assets")
-set(resource_dir "${CMAKE_BINARY_DIR}/share/clang-doc")
+if(MSVC)
----------------
ilovepi wrote:

Doesn't this reflect the compiler and not the build system/project structure? I'm pretty sure you can use Ninja and build w/ MSVC, right? In that case, I'm not sure this is the right approach. 

https://github.com/llvm/llvm-project/pull/97540


More information about the cfe-commits mailing list