Skip to content

Commit

Permalink
[LibWebRTC] Unreviewed, build fix for Linux after 276708@main
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=271696

CMake was failing because there was an attempt to set 'webrtc' when it
was not yet defined. Move faulting lines below, once 'webrtc' is already
defined.

* Source/ThirdParty/libwebrtc/CMakeLists.txt:

Canonical link: https://commits.webkit.org/276724@main
  • Loading branch information
dpino committed Mar 27, 2024
1 parent 4652010 commit b815830
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions Source/ThirdParty/libwebrtc/CMakeLists.txt
Expand Up @@ -2169,10 +2169,7 @@ else ()
Source/webrtc/modules/audio_device/linux/audio_mixer_manager_pulse_linux.cc
Source/webrtc/modules/audio_device/linux/pulseaudiosymboltable_linux.cc
)
target_include_directories(webrtc PRIVATE ${LibPulse_INCLUDE_DIRS})
target_compile_options(webrtc PRIVATE ${LibPulse_COMPILE_OPTIONS})
endif ()

endif ()

add_library(webrtc STATIC ${webrtc_SOURCES})
Expand Down Expand Up @@ -2303,6 +2300,10 @@ else ()
endif ()

target_include_directories(webrtc ${webrtc_INCLUDE_DIRECTORIES})
if (LIBPULSE_FOUND)
target_include_directories(webrtc PRIVATE ${LibPulse_INCLUDE_DIRS})
target_compile_options(webrtc PRIVATE ${LibPulse_COMPILE_OPTIONS})
endif ()

# libsrtp package compilation
set(libsrtp_SOURCES
Expand Down

0 comments on commit b815830

Please sign in to comment.