[libc-commits] [PATCH] D133960: [libc][cmake] separate installing headers

Michael Jones via Phabricator via libc-commits libc-commits at lists.llvm.org
Thu Sep 15 11:26:50 PDT 2022


michaelrj created this revision.
michaelrj added a reviewer: sivachandra.
Herald added subscribers: libc-commits, ecnelises, tschuett, mgorny.
Herald added projects: libc-project, All.
michaelrj requested review of this revision.

Now libc headers can be installed separately from installing the rest of
the libc.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D133960

Files:
  libc/CMakeLists.txt
  libc/include/CMakeLists.txt


Index: libc/include/CMakeLists.txt
===================================================================
--- libc/include/CMakeLists.txt
+++ libc/include/CMakeLists.txt
@@ -266,5 +266,5 @@
   get_filename_component(nested_dir ${relative_path} DIRECTORY)
   install(FILES ${header_file}
           DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/${nested_dir}
-          COMPONENT ${LIBC_COMPONENT})
+          COMPONENT libc-headers)
 endforeach()
Index: libc/CMakeLists.txt
===================================================================
--- libc/CMakeLists.txt
+++ libc/CMakeLists.txt
@@ -146,9 +146,14 @@
 set(LIBC_INSTALL_DEPENDS)
 set(LIBC_INSTALL_TARGET)
 if(LLVM_LIBC_FULL_BUILD)
+  add_llvm_install_targets(
+    install-libc-headers
+    DEPENDS libc-headers
+    COMPONENT libc-headers
+  )
   set(LIBC_TARGET libc)
   set(LIBC_COMPONENT libc)
-  set(LIBC_INSTALL_DEPENDS "libc;libc-headers;libc-startup")
+  set(LIBC_INSTALL_DEPENDS "libc;install-libc-headers;libc-startup")
   set(LIBC_INSTALL_TARGET install-libc)
   set(LIBC_ARCHIVE_NAME c)
 else()


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D133960.460468.patch
Type: text/x-patch
Size: 1057 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libc-commits/attachments/20220915/88a82707/attachment.bin>


More information about the libc-commits mailing list