Skip to content
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
11 changes: 10 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,19 +24,27 @@ jobs:
cc: gcc
cxx: g++
werror: ON
capi: ON
- name: Linux Clang
os: ubuntu-latest
cc: clang
cxx: clang++
werror: ON
capi: ON
- name: macOS AppleClang
os: macos-latest
werror: ON
capi: ON
# Warnings stay non-fatal on MSVC until /W4 output has been
# triaged (docs/PERFORMANCE.md "Known debt").
# triaged (docs/PERFORMANCE.md "Known debt"). capi stays OFF here too:
# tools/capi carries no __declspec(dllexport) (unlike DspTap's), so an
# MSVC build would link a DLL exporting nothing — it would pass without
# gating anything. Turn it ON in the same change that gives the C ABI
# an export decoration.
- name: Windows MSVC
os: windows-latest
werror: OFF
capi: OFF
steps:
- uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
with:
Expand All @@ -50,6 +58,7 @@ jobs:
cmake -B build
-DCMAKE_BUILD_TYPE=Release
-DSRT_WERROR=${{ matrix.werror }}
-DSRT_BUILD_CAPI=${{ matrix.capi }}

- name: Build
run: cmake --build build --config Release -j 4
Expand Down
5 changes: 5 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ add_subdirectory(submodules/dsptap)

add_library(SampleRateTap INTERFACE)
add_library(SampleRateTap::SampleRateTap ALIAS SampleRateTap)
# The family-wide alias, per the namespace convention in taphouse's README: one
# `tap::<library>` sub-namespace per repo, and the CMake alias matches it. The
# older spelling above stays so existing consumers keep working; prefer
# `tap::samplerate` in new code.
add_library(tap::samplerate ALIAS SampleRateTap)
target_include_directories(SampleRateTap INTERFACE
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
$<INSTALL_INTERFACE:include>)
Expand Down
Loading