Repository navigation
Conversation
yawaflua
left a comment
There was a problem hiding this comment.
Не очень понимаю нужду в использовании сторонней библиотеки для управления звуком.
Как по мне лучше абстрагироваться от использования сторонних библиотек для таких важных функций. Если же данная библиотека/программа используется только для AirPlay - есть смысл переписать коллы, нежели скачивать доп оборудование, за которое мы не можем нести ответственность. Открыто для диалога
|
Согласен, что ради громкости не хочется тянуть зависимость — но уточню, что там происходит, потому что это не библиотека.
Но с идеей абстракции согласен, и она лучше того, что в PR. Переделаю на общий exec-бэкенд, без упоминания Капсулы в коде:
Тогда ядро вообще не знает про конкретное устройство, а |
|
окей, но только я сделаю пуш не в мастер, а в стейдж ветку |
Adds an optional volume backend for the VK Capsula Mini. The device panel uses
capsule-ipc VOL, and that is the only path that keeps the panel, ALSA and AirPlay volume in sync.The existing amixer backend is untouched and still selected by
MARUSYA_MIXER_CONTROL; the new backend is opt-in withMARUSYA_CAPSULE_IPC=1.capsule-ipc GETreports the volume and mute state;VOLalso unmutes, like the panel;MUTEis a toggle, so SetMuted reads the current state first.Tested on a real Capsula Mini: 90 -> 30, 90 -> 80, 90 -> 100. Unit tests use a fake runner; the agent builds.