commit: afdcd6a6ce9c7dbe5e3a87bf6405363efb959c9e
Author: James Le Cuirot <chewi <AT> gentoo <DOT> org>
AuthorDate: Mon Aug 7 15:04:37 2023 +0000
Commit: James Le Cuirot <chewi <AT> gentoo <DOT> org>
CommitDate: Mon Aug 7 15:05:06 2023 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=afdcd6a6
games-fps/gzdoom: Backport my local vs system headers fix
Signed-off-by: James Le Cuirot <chewi <AT> gentoo.org>
games-fps/gzdoom/files/gzdoom-4.10.0-headers.patch | 106 +++++++++++++++++++++
games-fps/gzdoom/gzdoom-4.10.0.ebuild | 3 +-
2 files changed, 108 insertions(+), 1 deletion(-)
diff --git a/games-fps/gzdoom/files/gzdoom-4.10.0-headers.patch
b/games-fps/gzdoom/files/gzdoom-4.10.0-headers.patch
new file mode 100644
index 000000000000..3a3a153b5a6e
--- /dev/null
+++ b/games-fps/gzdoom/files/gzdoom-4.10.0-headers.patch
@@ -0,0 +1,106 @@
+From b95dbaf914618cccaeaa95c650e02be669e477f8 Mon Sep 17 00:00:00 2001
+From: James Le Cuirot <ch...@gentoo.org>
+Date: Sat, 1 Jul 2023 13:36:48 +0100
+Subject: [PATCH] Ensure local headers are used over system headers to avoid
+ breakage
+
+types.h was being picked up from webp rather than locally due to the
+`include_directories` call for GTK (and therefore its -I arguments)
+coming before the same call for the local sources. webp can be pulled in
+via GTK -> gdk-pixbuf -> tiff -> webp.
+
+This can be avoided by specifying `SYSTEM` or `BEFORE` as appropriate
+when calling `include_directories`. I have done both for good measure.
+---
+ src/CMakeLists.txt | 18 ++++++++++--------
+ tools/zipdir/CMakeLists.txt | 2 +-
+ 2 files changed, 11 insertions(+), 9 deletions(-)
+
+diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
+index cf254ad6a..b4d06f9ce 100644
+--- a/src/CMakeLists.txt
++++ b/src/CMakeLists.txt
+@@ -97,7 +97,7 @@ else()
+ if( NOT DYN_GTK )
+ set( PROJECT_LIBRARIES
${PROJECT_LIBRARIES} ${GTK3_LIBRARIES} )
+ endif()
+- include_directories( ${GTK3_INCLUDE_DIRS} )
++ include_directories( SYSTEM
${GTK3_INCLUDE_DIRS} )
+ link_directories( ${GTK3_LIBRARY_DIRS} )
+ else()
+ pkg_check_modules( GTK2 gtk+-2.0 )
+@@ -105,7 +105,7 @@ else()
+ if( NOT DYN_GTK )
+ set( PROJECT_LIBRARIES
${PROJECT_LIBRARIES} ${GTK2_LIBRARIES} )
+ endif()
+- include_directories(
${GTK2_INCLUDE_DIRS} )
++ include_directories( SYSTEM
${GTK2_INCLUDE_DIRS} )
+ link_directories( ${GTK2_LIBRARY_DIRS} )
+ else()
+ set( NO_GTK ON )
+@@ -132,7 +132,7 @@ else()
+ # Non-Windows version also needs SDL except native OS X backend
+ if( NOT APPLE OR NOT OSX_COCOA_BACKEND )
+ find_package( SDL2 REQUIRED )
+- include_directories( "${SDL2_INCLUDE_DIR}" )
++ include_directories( SYSTEM "${SDL2_INCLUDE_DIR}" )
+ set( PROJECT_LIBRARIES ${PROJECT_LIBRARIES} "${SDL2_LIBRARY}" )
+ endif()
+
+@@ -143,7 +143,7 @@ if( NOT NO_OPENAL )
+ find_package( OpenAL )
+ mark_as_advanced(CLEAR OPENAL_INCLUDE_DIR)
+ if( OPENAL_INCLUDE_DIR )
+- include_directories( ${OPENAL_INCLUDE_DIR} )
++ include_directories( SYSTEM ${OPENAL_INCLUDE_DIR} )
+ mark_as_advanced(CLEAR OPENAL_LIBRARY)
+ if( OPENAL_LIBRARY )
+ set( PROJECT_LIBRARIES ${OPENAL_LIBRARY}
${PROJECT_LIBRARIES} )
+@@ -373,17 +373,17 @@ endif()
+
+ if( VPX_FOUND )
+ add_definitions( "-DUSE_LIBVPX=1" )
+- include_directories( "${VPX_INCLUDE_DIR}" )
++ include_directories( SYSTEM "${VPX_INCLUDE_DIR}" )
+ set( PROJECT_LIBRARIES ${PROJECT_LIBRARIES} ${VPX_LIBRARIES} )
+ else()
+ message( SEND_ERROR "Could not find libvpx" )
+ endif()
+
+-include_directories( "${ZLIB_INCLUDE_DIR}" "${BZIP2_INCLUDE_DIR}"
"${LZMA_INCLUDE_DIR}" "${JPEG_INCLUDE_DIR}" "${ZMUSIC_INCLUDE_DIR}"
"${DRPC_INCLUDE_DIR}")
++include_directories( SYSTEM "${ZLIB_INCLUDE_DIR}" "${BZIP2_INCLUDE_DIR}"
"${LZMA_INCLUDE_DIR}" "${JPEG_INCLUDE_DIR}" "${ZMUSIC_INCLUDE_DIR}"
"${DRPC_INCLUDE_DIR}")
+
+ if( ${HAVE_VM_JIT} )
+ add_definitions( -DHAVE_VM_JIT )
+- include_directories( "${ASMJIT_INCLUDE_DIR}" )
++ include_directories( SYSTEM "${ASMJIT_INCLUDE_DIR}" )
+ set( PROJECT_LIBRARIES ${PROJECT_LIBRARIES} "${ASMJIT_LIBRARIES}")
+ endif()
+
+@@ -1243,7 +1243,9 @@ endif()
+
+ target_link_libraries( zdoom ${PROJECT_LIBRARIES} gdtoa lzma
${ZMUSIC_LIBRARIES} )
+
+-include_directories( .
++include_directories(
++ BEFORE
++ .
+ common/audio/sound
+ common/audio/music
+ common/2d
+diff --git a/tools/zipdir/CMakeLists.txt b/tools/zipdir/CMakeLists.txt
+index 65eb2fb72..762d9027f 100644
+--- a/tools/zipdir/CMakeLists.txt
++++ b/tools/zipdir/CMakeLists.txt
+@@ -1,7 +1,7 @@
+ cmake_minimum_required( VERSION 3.1.0 )
+
+ if( NOT CMAKE_CROSSCOMPILING )
+- include_directories( "${ZLIB_INCLUDE_DIR}" "${BZIP2_INCLUDE_DIR}"
"${LZMA_INCLUDE_DIR}" )
++ include_directories( SYSTEM "${ZLIB_INCLUDE_DIR}"
"${BZIP2_INCLUDE_DIR}" "${LZMA_INCLUDE_DIR}" )
+ add_executable( zipdir
+ zipdir.c )
+ target_link_libraries( zipdir ${ZLIB_LIBRARIES} ${BZIP2_LIBRARIES} lzma
)
+--
+2.41.0
+
diff --git a/games-fps/gzdoom/gzdoom-4.10.0.ebuild
b/games-fps/gzdoom/gzdoom-4.10.0.ebuild
index 16c223070e34..605c7cdcf993 100644
--- a/games-fps/gzdoom/gzdoom-4.10.0.ebuild
+++ b/games-fps/gzdoom/gzdoom-4.10.0.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2022 Gentoo Authors
+# Copyright 1999-2023 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2
EAPI=8
@@ -31,6 +31,7 @@ S="${WORKDIR}/${PN}-g${PV}"
PATCHES=(
"${FILESDIR}"/${PN}-4.7.1-Introduce-the-BUILD_NONFREE-option.patch
"${FILESDIR}"/${P}-gcc-13.patch
+ "${FILESDIR}"/${P}-headers.patch
)
src_prepare() {
