Skip to content

Commit 986f567

Browse files
committed
CMake: Eliminate duplicate include dirs
1 parent 0658134 commit 986f567

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

src/CMakeLists.txt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,12 +90,13 @@ FIND_PACKAGE(FFmpeg REQUIRED)
9090

9191
foreach(ffmpeg_comp AVCODEC AVDEVICE AVFORMAT AVFILTER AVUTIL POSTPROC SWSCALE SWRESAMPLE AVRESAMPLE)
9292
if(${ffmpeg_comp}_FOUND)
93-
# message(STATUS "Adding include dir for ${ffmpeg_comp}")
94-
include_directories(${${ffmpeg_comp}_INCLUDE_DIRS})
93+
list(APPEND FF_INCLUDES ${${ffmpeg_comp}_INCLUDE_DIRS})
9594
add_definitions(${${ffmpeg_comp}_DEFINITIONS})
9695
list(APPEND FF_LIBRARIES ${${ffmpeg_comp}_LIBRARIES})
9796
endif()
9897
endforeach()
98+
list(REMOVE_DUPLICATES FF_INCLUDES)
99+
include_directories(${FF_INCLUDES})
99100

100101
################# LIBOPENSHOT-AUDIO ###################
101102
# Find JUCE-based openshot Audio libraries

tests/CMakeLists.txt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,10 +84,12 @@ FIND_PACKAGE(FFmpeg REQUIRED)
8484
foreach(ffmpeg_comp AVCODEC AVDEVICE AVFORMAT AVFILTER AVUTIL POSTPROC SWSCALE SWRESAMPLE AVRESAMPLE)
8585
if(${ffmpeg_comp}_FOUND)
8686
# Include FFmpeg headers (needed for compile)
87-
include_directories(${${ffmpeg_comp}_INCLUDE_DIRS})
87+
list(APPEND FF_INCLUDES ${${ffmpeg_comp}_INCLUDE_DIRS})
8888
add_definitions(${${ffmpeg_comp}_DEFINITIONS})
8989
endif()
9090
endforeach()
91+
list(REMOVE_DUPLICATES FF_INCLUDES)
92+
include_directories(${FF_INCLUDES})
9193

9294
################# LIBOPENSHOT-AUDIO ###################
9395
# Find JUCE-based openshot Audio libraries
@@ -101,12 +103,14 @@ include_directories(${LIBOPENSHOT_AUDIO_INCLUDE_DIRS})
101103
foreach(qt_lib Qt5Widgets Qt5Core Qt5Gui Qt5Multimedia Qt5MultimediaWidgets)
102104
find_package(${qt_lib} REQUIRED)
103105
# Header files
104-
include_directories(${${qt_lib}_INCLUDE_DIRS})
106+
list(APPEND QT_INCLUDES ${${qt_lib}_INCLUDE_DIRS})
105107
# Compiler definitions
106108
add_definitions(${${qt_lib}_DEFINITIONS})
107109
# Other CFLAGS
108110
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${${qt_lib}_EXECUTABLE_COMPILE_FLAGS}")
109111
endforeach()
112+
list(REMOVE_DUPLICATES QT_INCLUDES)
113+
include_directories(${QT_INCLUDES})
110114

111115
# Manually moc Qt files
112116
qt5_wrap_cpp(MOC_FILES ${QT_HEADER_FILES})

0 commit comments

Comments
 (0)