From a77e5889bc43e4d6589a6f4784b2f9a4488a8839 Mon Sep 17 00:00:00 2001 From: Szabolcs Hubai Date: Sun, 7 Dec 2025 21:04:13 +0100 Subject: [PATCH] snapcast: define ALSA support explicitly As Hannu Nyman pointed out, in some cases the build of the package is able to fail: Package snapserver is missing dependencies for the following libraries: libasound.so.2 He said: on some build runs, alsa is built (as required by other packages), and if it is built before snapcast, it may be detected, if the feature is not explicitly disabled in CMakeLists.txt. Also: "I think that you could drop that AUDIO_SUPPORT conditional. AUDIO_SUPPORT is selected e.g. if USB_SUPPORT is selected, so it is not that big restriction." Suggested-by: Hannu Nyman Signed-off-by: Szabolcs Hubai --- sound/snapcast/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/snapcast/Makefile b/sound/snapcast/Makefile index b097bad2f5..20b57b7f61 100644 --- a/sound/snapcast/Makefile +++ b/sound/snapcast/Makefile @@ -21,6 +21,7 @@ include $(INCLUDE_DIR)/cmake.mk CMAKE_OPTIONS += \ -DBUILD_TESTS=OFF \ + -DBUILD_WITH_ALSA=ON \ -DBUILD_WITH_AVAHI=OFF \ -DBUILD_WITH_PULSE=OFF @@ -28,7 +29,7 @@ define Package/snapcast/Default SECTION:=sound CATEGORY:=Sound TITLE:=Synchronous multiroom audio player - DEPENDS:=+AUDIO_SUPPORT:alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr + DEPENDS:=+alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr URL:=https://github.com/badaix/snapcast endef