Skip to content

Conversation

@vonosmas
Copy link
Contributor

Add dependency on headers with in_addr and in_addr_t type definitions to ensure that these headers will be properly installed by "install-libc" CMake target.

Add dependency on headers with `in_addr` and `in_addr_t`
type definitions to ensure that these headers will be properly
installed by "install-libc" CMake target.
@vonosmas vonosmas requested a review from c8ef November 20, 2025 21:57
@llvmbot llvmbot added the libc label Nov 20, 2025
@vonosmas vonosmas enabled auto-merge (squash) November 20, 2025 21:58
@llvmbot
Copy link
Member

llvmbot commented Nov 20, 2025

@llvm/pr-subscribers-libc

Author: Alexey Samsonov (vonosmas)

Changes

Add dependency on headers with in_addr and in_addr_t type definitions to ensure that these headers will be properly installed by "install-libc" CMake target.


Full diff: https://github.com/llvm/llvm-project/pull/168951.diff

1 Files Affected:

  • (modified) libc/include/CMakeLists.txt (+2)
diff --git a/libc/include/CMakeLists.txt b/libc/include/CMakeLists.txt
index 7b52daf9a8062..96c3a88f3fcc8 100644
--- a/libc/include/CMakeLists.txt
+++ b/libc/include/CMakeLists.txt
@@ -200,6 +200,8 @@ add_header_macro(
   DEPENDS
     .llvm_libc_common_h
     .inttypes
+    .llvm-libc-types.in_addr
+    .llvm-libc-types.in_addr_t
 )
 
 file(MAKE_DIRECTORY ${LIBC_INCLUDE_DIR}/netinet)

@vonosmas vonosmas merged commit 1136239 into llvm:main Nov 20, 2025
26 of 27 checks passed
@github-actions
Copy link

🐧 Linux x64 Test Results

The build succeeded and no tests ran. This is expected in some build configurations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants