From: Michael Shigorin <mike@altlinux.org>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] feature use/sound/alsa
Date: Sun, 7 Jan 2018 18:17:54 +0300
Message-ID: <20180107151754.GH9899@imap.altlinux.org> (raw)
In-Reply-To: <cd5e11cf-158c-fce3-3189-2174d0c6eaaf@ya.ru>
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
next prev parent reply other threads:[~2018-01-07 15:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-06 9:22 Антон Мидюков
2018-01-07 15:17 ` Michael Shigorin [this message]
2018-01-08 9:11 ` Антон Мидюков
2018-01-09 12:39 ` [devel-distro] m-p ошибка при сборке regular-icewm.iso Oleg
2018-01-09 12:44 ` Антон Мидюков
2018-01-10 7:22 ` Антон Мидюков
2018-01-09 13:21 ` Michael Shigorin
2018-01-10 8:35 ` Oleg
2018-01-11 8:03 ` Michael Shigorin
2018-01-21 14:25 ` Антон Мидюков
2018-01-18 11:44 ` [devel-distro] feature use/sound/alsa Michael Shigorin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180107151754.GH9899@imap.altlinux.org \
--to=mike@altlinux.org \
--cc=devel-distro@lists.altlinux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
ALT Linux Distributions development
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel-distro/0 devel-distro/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel-distro devel-distro/ http://lore.altlinux.org/devel-distro \
devel-distro@lists.altlinux.org devel-distro@lists.altlinux.ru devel-distro@lists.altlinux.com
public-inbox-index devel-distro
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel-distro
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git