Skip to content

Commit 7d9808b

Browse files
committed
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 <[email protected]> Signed-off-by: Szabolcs Hubai <[email protected]>
1 parent 026d608 commit 7d9808b

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

sound/snapcast/Makefile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,15 @@ include $(INCLUDE_DIR)/cmake.mk
2121

2222
CMAKE_OPTIONS += \
2323
-DBUILD_TESTS=OFF \
24+
-DBUILD_WITH_ALSA=ON \
2425
-DBUILD_WITH_AVAHI=OFF \
2526
-DBUILD_WITH_PULSE=OFF
2627

2728
define Package/snapcast/Default
2829
SECTION:=sound
2930
CATEGORY:=Sound
3031
TITLE:=Synchronous multiroom audio player
31-
DEPENDS:=+AUDIO_SUPPORT:alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr
32+
DEPENDS:=+alsa-lib +libexpat +libatomic +libogg +libflac +libopus +boost +libsoxr
3233
URL:=https://github.com/badaix/snapcast
3334
endef
3435

0 commit comments

Comments
 (0)