[libcxx-commits] [libcxx] 3c8397f - [libc++] Fix installation path for the modulemap

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jan 23 17:12:41 PST 2023


Author: Louis Dionne
Date: 2023-01-23T20:12:26-05:00
New Revision: 3c8397fa83fe44e1a88543601e777889affa8e07

URL: https://github.com/llvm/llvm-project/commit/3c8397fa83fe44e1a88543601e777889affa8e07
DIFF: https://github.com/llvm/llvm-project/commit/3c8397fa83fe44e1a88543601e777889affa8e07.diff

LOG: [libc++] Fix installation path for the modulemap

As pointed out in a post-commit comment of https://reviews.llvm.org/rGd00e035a4270.

Differential Revision: https://reviews.llvm.org/D142365

Added: 
    

Modified: 
    libcxx/include/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/libcxx/include/CMakeLists.txt b/libcxx/include/CMakeLists.txt
index 5d4fa911865f1..51bfc8ca53658 100644
--- a/libcxx/include/CMakeLists.txt
+++ b/libcxx/include/CMakeLists.txt
@@ -909,13 +909,18 @@ if (LIBCXX_INSTALL_HEADERS)
     )
   endforeach()
 
-  # Install the generated __config_site and the generated modulemap file.
+  # Install the generated __config_site file to the per-target include dir.
   install(FILES "${LIBCXX_GENERATED_INCLUDE_TARGET_DIR}/__config_site"
-                "${LIBCXX_GENERATED_INCLUDE_DIR}/module.modulemap"
     DESTINATION "${LIBCXX_INSTALL_INCLUDE_TARGET_DIR}"
     PERMISSIONS OWNER_READ OWNER_WRITE GROUP_READ WORLD_READ
     COMPONENT cxx-headers)
 
+  # Install the generated modulemap file to the generic include dir.
+  install(FILES "${LIBCXX_GENERATED_INCLUDE_DIR}/module.modulemap"
+    DESTINATION "${LIBCXX_INSTALL_INCLUDE_DIR}"
+    PERMISSIONS OWNER_READ OWNER_WRITE GROUP_READ WORLD_READ
+    COMPONENT cxx-headers)
+
   if (NOT CMAKE_CONFIGURATION_TYPES)
     add_custom_target(install-cxx-headers
                       DEPENDS cxx-headers


        


More information about the libcxx-commits mailing list