From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sun, 7 Jan 2018 18:17:54 +0300 From: Michael Shigorin To: devel-distro@lists.altlinux.org Message-ID: <20180107151754.GH9899@imap.altlinux.org> References: MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23.88.hg577987ca2d02 (2014-03-12) Subject: Re: [devel-distro] feature use/sound/alsa X-BeenThere: devel-distro@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: Distributions development List-Id: Distributions development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Jan 2018 15:17:54 -0000 Archived-At: List-Archive: On Sat, Jan 06, 2018 at 04:22:27PM +0700, Антон Мидюков wrote: > 1. Предлагаю в фичу use/sound/alsa (+alsa) добавить пакет udev-alsa, > который чинит пропадающий уровень громкости при перезагрузке на системах > без pulseaudio. В этой фиче сейчас только apulse. Смотреть первый > приложенный патч. ack, только хорошо бы подобное описание и в патче приводить; поправил ровно в этом аспекте. > 2. Предлагаю добавить эту фичу всем десктопным дистрибутивам без > pulseaudio. Ну, а регуляркам на Cinnamon и gnome3 добавить фичу +pulse. > Если считаете, что пульса на гноме и циннамоне не нужна, то бог с ней, > давайте аналогично добавим им +alsa. Смотреть второй патч. nak; добавлять лучше сразу в distro/.regular-sysv, как мне кажется: если кому когда захочется именно регулярообразную исошку с sysv и pulseaudio, вполне можно описать отдельно. В производные от distro/.regular-sysv-gtk такая правка попадёт автоматом, там отдельно не надо писать (такое бывает удобней замечать с REPORT=1 и доступным graphviz, вот только инструмент для анализа сразу нескольких образов не был реализован в первую очередь по причине отсутствия представления о том, каким бы хотелось видеть результат). В distro/.regular-install-x11-full, как мне кажется, тоже стоит добавить (и соответственно сократить в обоих производных из предложенной правки). В остальном похоже на правду. > -distro/regular-icewm: distro/.regular-sysv-gtk +icewm +nm \ > +distro/regular-icewm: distro/.regular-sysv-gtk +icewm +nm +alsa \ > -distro/regular-wmaker: distro/.regular-sysv \ > +distro/regular-wmaker: distro/.regular-sysv +alsa \ > -distro/regular-xfce: distro/.regular-gtk mixin/regular-xfce \ > +distro/regular-xfce: distro/.regular-gtk mixin/regular-xfce +alsa \ > -distro/regular-sysv-tde: distro/.regular-install-x11-full mixin/regular-tde > +distro/regular-sysv-tde: distro/.regular-install-x11-full mixin/regular-tde +alsa Предлагаю сократить патч и прислать заново. PS: вообще здесь напрашивается переключатель вроде того, какие были сделаны в фичах bootloader, browser, init; возможно, подход к переключателям стоит обобщить и как-то подстраховать от документированной проблемы с попыткой повторного выставления уже фигурировавшего в цепочке зависимостей варианта (features.in/bootloader/README). --  ---- WBR, Michael Shigorin / http://altlinux.org   ------ http://opennet.ru / http://anna-news.info