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

Remove legacy_check CMake macro #11248

Merged
merged 1 commit into from
Sep 11, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 0 additions & 2 deletions UI/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_subdirectory(obs-frontend-api)

option(ENABLE_UI "Enable building with UI (requires Qt)" ON)
Expand Down
2 changes: 0 additions & 2 deletions UI/frontend-plugins/aja-output-ui/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(NOT ENABLE_AJA)
target_disable(aja-output-ui)
return()
Expand Down
2 changes: 0 additions & 2 deletions UI/frontend-plugins/decklink-captions/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(NOT ENABLE_DECKLINK)
target_disable(decklink-captions)
return()
Expand Down
2 changes: 0 additions & 2 deletions UI/frontend-plugins/decklink-output-ui/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(NOT ENABLE_DECKLINK)
target_disable(decklink-output-ui)
return()
Expand Down
2 changes: 0 additions & 2 deletions UI/frontend-plugins/frontend-tools/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

find_package(Qt6 REQUIRED Widgets)

if(OS_LINUX OR OS_FREEBSD OR OS_OPENBSD)
Expand Down
2 changes: 0 additions & 2 deletions UI/obs-frontend-api/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_library(obs-frontend-api SHARED)
add_library(OBS::frontend-api ALIAS obs-frontend-api)

Expand Down
2 changes: 0 additions & 2 deletions UI/win-update/updater/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

find_package(zstd)
find_package(nlohmann_json 3 REQUIRED)

Expand Down
7 changes: 0 additions & 7 deletions cmake/common/helpers_common.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -435,13 +435,6 @@ function(check_uuid uuid_string return_value)
set(${return_value} ${valid_uuid} PARENT_SCOPE)
endfunction()

# legacy_check: Check if new CMake framework was not enabled and load legacy rules instead
macro(legacy_check)
if(OBS_CMAKE_VERSION VERSION_LESS 3.0.0)
message(FATAL_ERROR "CMake version changed between CMakeLists.txt.")
endif()
endmacro()

# add_obs_plugin: Add plugin subdirectory if host platform is in specified list of supported platforms and architectures
function(add_obs_plugin target)
set(options WITH_MESSAGE)
Expand Down
4 changes: 0 additions & 4 deletions cmake/windows/architecture.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,6 @@ if(OBS_PARENT_ARCHITECTURE STREQUAL CMAKE_GENERATOR_PLATFORM)
)
endif()
else()
# legacy_check: Stub macro for child architecture builds
macro(legacy_check)
endmacro()

# target_disable_feature: Stub macro for child architecture builds
macro(target_disable_feature)
endmacro()
Expand Down
2 changes: 0 additions & 2 deletions deps/w32-pthreads/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

add_library(w32-pthreads SHARED EXCLUDE_FROM_ALL)
add_library(OBS::w32-pthreads ALIAS w32-pthreads)

Expand Down
2 changes: 0 additions & 2 deletions libobs-opengl/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_library(libobs-opengl SHARED)
add_library(OBS::libobs-opengl ALIAS libobs-opengl)

Expand Down
2 changes: 0 additions & 2 deletions libobs-winrt/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

add_library(libobs-winrt-headers INTERFACE)
add_library(OBS::winrt-headers ALIAS libobs-winrt-headers)

Expand Down
2 changes: 0 additions & 2 deletions libobs/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

include(cmake/obs-version.cmake)

if(OS_WINDOWS AND NOT OBS_PARENT_ARCHITECTURE STREQUAL CMAKE_GENERATOR_PLATFORM)
Expand Down
2 changes: 0 additions & 2 deletions plugins/aja/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_AJA "Build OBS with aja support" ON)

if(NOT ENABLE_AJA)
Expand Down
2 changes: 0 additions & 2 deletions plugins/coreaudio-encoder/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

if(OS_WINDOWS)
option(ENABLE_COREAUDIO_ENCODER "Enable building with CoreAudio encoder (Windows)" ON)
if(NOT ENABLE_COREAUDIO_ENCODER)
Expand Down
2 changes: 0 additions & 2 deletions plugins/decklink/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_DECKLINK "Build OBS with Decklink support" ON)
if(NOT ENABLE_DECKLINK)
target_disable(decklink)
Expand Down
2 changes: 0 additions & 2 deletions plugins/image-source/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_library(image-source MODULE)
add_library(OBS::image-source ALIAS image-source)

Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-alsa/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_ALSA "Build OBS with ALSA support" ON)

if(NOT ENABLE_ALSA)
Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-capture/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

find_package(X11 REQUIRED)

find_package(
Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-jack/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_JACK "Build OBS with JACK support" OFF)
if(NOT ENABLE_JACK)
target_disable(linux-jack)
Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-pipewire/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_PIPEWIRE "Enable PipeWire support" ON)
if(NOT ENABLE_PIPEWIRE)
target_disable(linux-pipewire)
Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-pulseaudio/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(NOT ENABLE_PULSEAUDIO)
target_disable(linux-pulseaudio)
return()
Expand Down
2 changes: 0 additions & 2 deletions plugins/linux-v4l2/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_V4L2 "Build OBS with v4l2 support" ON)
option(ENABLE_UDEV "Build linux-v4l2 with UDEV support" ON)

Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-ffmpeg/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_FFMPEG_LOGGING "Enables obs-ffmpeg logging" OFF)
option(ENABLE_NEW_MPEGTS_OUTPUT "Use native SRT/RIST mpegts output" ON)

Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-ffmpeg/ffmpeg-mux/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_FFMPEG_MUX_DEBUG "Enable FFmpeg-mux debugging" OFF)

find_package(FFmpeg REQUIRED COMPONENTS avcodec avutil avformat)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-ffmpeg/obs-amf-test/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

find_package(AMF 1.4.29 REQUIRED)

add_executable(obs-amf-test)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-filters/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_library(obs-filters MODULE)
add_library(OBS::filters ALIAS obs-filters)

Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-libfdk/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_LIBFDK "Enable FDK AAC support" OFF)
if(NOT ENABLE_LIBFDK)
target_disable(obs-libfdk)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-outputs/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

find_package(MbedTLS REQUIRED)
find_package(ZLIB REQUIRED)

Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-qsv11/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_QSV11 "Build Intel QSV11 Hardware Encoder." TRUE)
if(NOT ENABLE_QSV11)
target_disable_feature(obs-qsv11 "Intel QSV11 Hardware Encoder")
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-qsv11/obs-qsv-test/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.24...3.25)

legacy_check()

add_executable(obs-qsv-test)

find_package(VPL 2.6 REQUIRED)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-transitions/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

add_library(obs-transitions MODULE)
add_library(OBS::transition ALIAS obs-transitions)

Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-vst/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_VST "Enable building OBS with VST plugin" ON)

if(NOT ENABLE_VST)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-webrtc/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_WEBRTC "Enable WebRTC Output support" ON)
if(NOT ENABLE_WEBRTC)
target_disable(obs-webrtc)
Expand Down
2 changes: 0 additions & 2 deletions plugins/obs-x264/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

find_package(Libx264 REQUIRED)

if(NOT TARGET OBS::opts-parser)
Expand Down
2 changes: 0 additions & 2 deletions plugins/oss-audio/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_OSS "Enable building with OSS audio support" ON)

if(NOT ENABLE_OSS)
Expand Down
2 changes: 0 additions & 2 deletions plugins/rtmp-services/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_SERVICE_UPDATES "Checks for service updates" ON)

set(
Expand Down
2 changes: 0 additions & 2 deletions plugins/sndio/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_SNDIO "Build OBS with sndio support" OFF)
if(NOT ENABLE_SNDIO)
target_disable(sndio)
Expand Down
2 changes: 0 additions & 2 deletions plugins/text-freetype2/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_FREETYPE "Enable FreeType text plugin" ON)

if(NOT ENABLE_FREETYPE)
Expand Down
2 changes: 0 additions & 2 deletions plugins/vlc-video/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

macro(check_vlc_path)
find_package(PkgConfig QUIET)
if(PKG_CONFIG_FOUND)
Expand Down
2 changes: 0 additions & 2 deletions shared/obs-scripting/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(NOT ENABLE_SCRIPTING)
target_disable_feature(obs-scripting "Scripting support")
return()
Expand Down
2 changes: 0 additions & 2 deletions shared/obs-scripting/obslua/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(POLICY CMP0078)
cmake_policy(SET CMP0078 NEW)
endif()
Expand Down
2 changes: 0 additions & 2 deletions shared/obs-scripting/obspython/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

if(POLICY CMP0078)
cmake_policy(SET CMP0078 NEW)
endif()
Expand Down
2 changes: 0 additions & 2 deletions test/test-input/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
cmake_minimum_required(VERSION 3.22...3.25)

legacy_check()

option(ENABLE_TEST_INPUT "Build test sources" OFF)

if(NOT ENABLE_TEST_INPUT)
Expand Down