Skip to content

Commit

Permalink
Fix ffpmeg/opencv dependencies. Enable C lang on Mac, for external pr…
Browse files Browse the repository at this point in the history
…ojects
  • Loading branch information
srcejon committed Sep 13, 2023
1 parent e83d974 commit df4c6ca
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ cmake_minimum_required(VERSION 3.13.0)
set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>DLL")

if(APPLE)
project(sdrangel LANGUAGES CXX OBJCXX)
project(sdrangel LANGUAGES C CXX OBJCXX)
else()
project(sdrangel)
endif()
Expand Down
7 changes: 4 additions & 3 deletions external/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -185,9 +185,10 @@ else()
set(USE_PRECOMPILED_LIBS TRUE)
endif()

# We don't CACHE the _DEPENDS variables, as the external project is only added when NOT _FOUND
# and not in subsequent configures when _FOUND
# Use PARENT_SCOPE so they can still be used in other CMakeFiles.
# We don't CACHE the _DEPENDS variables, as some external projects are only added when NOT _FOUND
# and not in subsequent configures when _FOUND (E.g. ffmpeg/opencv aren't added second time
# around as it slows down the build and causes some dependencies to always be rebuilt.)
# Use PARENT_SCOPE so they can be used in other CMakeFiles.
macro(set_global NAME VALUE)
set(${NAME} ${VALUE})
set(${NAME} ${VALUE} PARENT_SCOPE)
Expand Down
4 changes: 2 additions & 2 deletions plugins/channelrx/demoddatv/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,8 @@ add_executable(ldpctool
)
install(TARGETS ldpctool DESTINATION ${INSTALL_BIN_DIR})

if(FFMPEG_EXTERNAL)
add_dependencies(${TARGET_NAME} ffmpeg)
if(DEFINED FFMPEG_DEPENDS)
add_dependencies(${TARGET_NAME} ${FFMPEG_DEPENDS})
endif()

install(TARGETS ${TARGET_NAME} DESTINATION ${INSTALL_FOLDER})
Expand Down
2 changes: 1 addition & 1 deletion plugins/channeltx/modatv/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ target_link_libraries(${TARGET_NAME}
)

if(DEFINED OpenCV_DEPENDS)
add_dependencies(${TARGET_NAME} opencv)
add_dependencies(${TARGET_NAME} ${OpenCV_DEPENDS})
endif()

install(TARGETS ${TARGET_NAME} DESTINATION ${INSTALL_FOLDER})
Expand Down
4 changes: 2 additions & 2 deletions plugins/channeltx/moddatv/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ target_link_libraries(${TARGET_NAME}
${SWRESAMPLE_LIBRARIES}
)

if(FFMPEG_EXTERNAL)
add_dependencies(${TARGET_NAME} ffmpeg)
if(DEFINED FFMPEG_DEPENDS)
add_dependencies(${TARGET_NAME} ${FFMPEG_DEPENDS})
endif()

install(TARGETS ${TARGET_NAME} DESTINATION ${INSTALL_FOLDER})
Expand Down

0 comments on commit df4c6ca

Please sign in to comment.