Skip to content

PR23 forest: Wrap long default_option calls #39

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.

Sign up for GitHub

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

Open
0cwa wants to merge 1 commit into
base: pr23/forest/base
Choose a base branch
from
+30 −6
Open

PR23 forest: Wrap long default_option calls #39

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 30 additions & 6 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1015,7 +1015,11 @@ if(MSVC)
else()
message(STATUS "Could NOT find sccache (missing executable)")
endif()
default_option(SCCACHE_SUPPORT "Enable sccache support" "SCCACHE_EXECUTABLE;CMAKE_DISABLE_PRECOMPILE_HEADERS")
default_option(
SCCACHE_SUPPORT
"Enable sccache support"
"SCCACHE_EXECUTABLE;CMAKE_DISABLE_PRECOMPILE_HEADERS"
)
message(STATUS "Support for sccache: ${SCCACHE_SUPPORT}")
if(SCCACHE_SUPPORT)
if(NOT CMAKE_DISABLE_PRECOMPILE_HEADERS)
Expand All @@ -1035,7 +1039,11 @@ else()
else()
message(STATUS "Could NOT find ccache (missing executable)")
endif()
default_option(CCACHE_SUPPORT "Enable ccache support" "CCACHE_EXECUTABLE;NOT ANDROID")
default_option(
CCACHE_SUPPORT
"Enable ccache support"
"CCACHE_EXECUTABLE;NOT ANDROID"
)

if(NOT DEFINED CMAKE_DISABLE_PRECOMPILE_HEADERS)
set(CMAKE_DISABLE_PRECOMPILE_HEADERS ${CCACHE_SUPPORT})
Expand Down Expand Up @@ -1101,7 +1109,11 @@ if(NOT MSVC)
if(MOLD_SYMLINK)
set(MOLD_SYMLINK_FOUND TRUE)
endif()
default_option(MOLD_SUPPORT "Use 'mold' for linking" "MOLD_FUSE_FOUND OR MOLD_SYMLINK_FOUND")
default_option(
MOLD_SUPPORT
"Use 'mold' for linking"
"MOLD_FUSE_FOUND OR MOLD_SYMLINK_FOUND"
)
if(MOLD_SUPPORT AND NOT ANDROID)
if(MOLD_FUSE_FOUND)
message(STATUS "Selecting mold as linker")
Expand Down Expand Up @@ -1141,7 +1153,11 @@ if(NOT MSVC)
endif()
endif()
# LLD 10.0.0 does not work because of https://bugs.llvm.org/show_bug.cgi?id=45769
default_option(LLD_SUPPORT "Use 'ld.lld' for linking" "LLD_VERSION VERSION_GREATER 10.0.0")
default_option(
LLD_SUPPORT
"Use 'ld.lld' for linking"
"LLD_VERSION VERSION_GREATER 10.0.0"
)
if(LLD_SUPPORT)
if(LLD_VERSION_STRING)
if(LLD_VERSION VERSION_GREATER 10.0.0)
Expand Down Expand Up @@ -3775,7 +3791,11 @@ if(PIPEWIRE)
endif()

# PortMidi
default_option(PORTMIDI "Enable the PortMidi backend for MIDI controllers" "NOT ANDROID")
default_option(
PORTMIDI
"Enable the PortMidi backend for MIDI controllers"
"NOT ANDROID"
)
if(PORTMIDI)
target_compile_definitions(mixxx-lib PUBLIC __PORTMIDI__)
find_package(PortMidi REQUIRED)
Expand Down Expand Up @@ -4955,7 +4975,11 @@ find_package(MP4)
find_package(MP4v2)
# It is enabled by default on Linux only, because other targets have other
# solutions. It requires MP4 or MP4v2.
default_option(FAAD "FAAD AAC audio file decoder support" "UNIX;NOT APPLE;MP4_FOUND OR MP4v2_FOUND")
default_option(
FAAD
"FAAD AAC audio file decoder support"
"UNIX;NOT APPLE;MP4_FOUND OR MP4v2_FOUND"
)
if(FAAD)
if(NOT MP4_FOUND AND NOT MP4v2_FOUND)
message(
Expand Down
Loading