From 1d1cc9ae18f677201bf7e47928c30562df967756 Mon Sep 17 00:00:00 2001 From: Green Sky Date: Wed, 17 Apr 2024 12:32:45 +0200 Subject: [PATCH] cmake: Fix an error in the declaration of target include directories Properly declare target include directories for generated includes. Previously the non targeted `include_directories()` was used, which had issue when using the `png_static` target in a submodule. Signed-off-by: Cosmin Truta --- CMakeLists.txt | 6 ++++-- scripts/cmake/AUTHORS.md | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index e0d0369501..ce8c054a0f 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -363,8 +363,6 @@ else() message(STATUS "Could not find an AWK-compatible program") endif() -include_directories(${CMAKE_CURRENT_BINARY_DIR}) - if(NOT AWK OR ANDROID OR IOS) # No awk available to generate sources; use pre-built pnglibconf.h configure_file(${CMAKE_CURRENT_SOURCE_DIR}/scripts/pnglibconf.h.prebuilt @@ -715,6 +713,8 @@ if(PNG_SHARED) endif() target_include_directories(png_shared PUBLIC $) + target_include_directories(png_shared + PUBLIC $) target_include_directories(png_shared SYSTEM INTERFACE $) target_link_libraries(png_shared PUBLIC ZLIB::ZLIB ${M_LIBRARY}) @@ -729,6 +729,8 @@ if(PNG_STATIC) DEBUG_POSTFIX "${PNG_DEBUG_POSTFIX}") target_include_directories(png_static PUBLIC $) + target_include_directories(png_static + PUBLIC $) target_include_directories(png_static SYSTEM INTERFACE $) target_link_libraries(png_static PUBLIC ZLIB::ZLIB ${M_LIBRARY}) diff --git a/scripts/cmake/AUTHORS.md b/scripts/cmake/AUTHORS.md index b4b61cfc0e..c098217865 100644 --- a/scripts/cmake/AUTHORS.md +++ b/scripts/cmake/AUTHORS.md @@ -21,6 +21,7 @@ Author List * Dan Rosser * David Callu * Eric Riff + * Erik Scholz * Gianfranco Costamagna * Gleb Mazovetskiy * Glenn Randers-Pehrson