shithub: opus

Download patch

ref: ce9d83be806941d48dc37034c0c6522a4bf09bb6
parent: 97fca1b3392c4144891b92d2a8c1a27389048873
author: Davide Beatrici <git@davidebeatrici.dev>
date: Thu May 28 06:53:18 EDT 2020

cmake/CFeatureCheck.cmake: fix feature tests failing when Opus is a submodule

CMAKE_SOURCE_DIR corresponds to the top project's source directory.
CMAKE_BINARY_DIR corresponds to the top project's binary directory.

The usage of these variables doesn't cause any problems when Opus is built as a standalone project.

This is not the case when Opus is added as submodule: the variables are set by the project that calls "add_subdirectory()".

The fix consists in using PROJECT_SOURCE_DIR and PROJECT_BINARY_DIR, which always refer to the current project.

Signed-off-by: Mark Harris <mark.hsj@gmail.com>

--- a/cmake/CFeatureCheck.cmake

+++ b/cmake/CFeatureCheck.cmake

@@ -27,7 +27,7 @@

   if (NOT DEFINED COMPILE_${FEATURE})

       message(STATUS "Performing Test ${FEATURE}")

-      try_compile(COMPILE_${FEATURE} ${CMAKE_BINARY_DIR} ${CMAKE_SOURCE_DIR}/cmake/${FILE}.c)

+      try_compile(COMPILE_${FEATURE} ${PROJECT_BINARY_DIR} ${PROJECT_SOURCE_DIR}/cmake/${FILE}.c)

   endif()

   if(COMPILE_${FEATURE})

@@ -36,4 +36,4 @@

   else()

     message(STATUS "Performing Test ${FEATURE} -- failed to compile")

   endif()

-endfunction()

\ No newline at end of file

+endfunction()