bootstrap-fix-include-next
authorLLVM Packaging Team <pkg-llvm-team@lists.alioth.debian.org>
Tue, 14 Jan 2025 13:16:47 +0000 (14:16 +0100)
committerSylvestre Ledru <sylvestre@debian.org>
Tue, 14 Jan 2025 13:16:47 +0000 (14:16 +0100)
When doing a bootstrap, we use a newly built clang.
When this one is used, if already installed on the system,
we have clang header in two places:
llvm-toolchain-7-7/build-llvm/lib/clang/7.0.0/include/inttypes.h
and
/usr/include/clang/7.0.0/include/inttypes.h

Because clang expects only one of his headers to be available, it uses
include_next to get the glibc (libc6-dev package) header.

However, in the previous example, because we have inttypes.h twice in the
include search path, clang's header will call itself without any effect.
Therefore, it will do include_next until the define from the libc is existing (ex: _INTTYPES_H)

Gbp-Pq: Name bootstrap-fix-include-next.diff

clang/lib/Headers/inttypes.h
clang/lib/Headers/limits.h

index 5150d22f8b2e4ed1cae2d738c8a369541ec40cc5..3e06ec3ce43819c9d5366827278248e7ec252577 100644 (file)
@@ -6,7 +6,7 @@
  *
 \*===----------------------------------------------------------------------===*/
 
-#ifndef __CLANG_INTTYPES_H
+#if !defined(__CLANG_INTTYPES_H) || !defined(_INTTYPES_H)
 // AIX system headers need inttypes.h to be re-enterable while _STD_TYPES_T
 // is defined until an inclusion of it without _STD_TYPES_T occurs, in which
 // case the header guard macro is defined.
index 56dffe568486cc597243d11184321fcbb86230f1..23d0bcd84d834f9b4b4f6d30191f960ee15244eb 100644 (file)
@@ -6,7 +6,7 @@
  *
 \*===----------------------------------------------------------------------===*/
 
-#ifndef __CLANG_LIMITS_H
+#if !defined(__CLANG_LIMITS_H) || !defined(_LIBC_LIMITS_H_)
 #define __CLANG_LIMITS_H
 
 #if defined(__MVS__) && __has_include_next(<limits.h>)