Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow RtAudio to be installed with FetchContent #440

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

witte
Copy link

@witte witte commented Oct 3, 2024

This is just to fix a little inconsistency that may put the RtAudioConfig.cmake and RtAudioConfig-version.cmake in the wrong folders when installing: when using RtAudio as a submodule or with CMake's FetchContent.

CMAKE_CURRENT_BINARY_DIR is already being used in other places in the same CMakeLists.txt file (see from

file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/RtAudioConfig.cmake.in "@PACKAGE_INIT@\n")
on).

Thanks!

@witte witte force-pushed the fix-RtAudioConfig.cmake-path branch from 2f32416 to 5dad9be Compare October 22, 2024 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant