[devel-distro] Переключатель между alsa и pulseaudio

[devel-distro] Переключатель между alsa и pulseaudio

[Антон Мидюков]

[-- Attachment #1: Type: text/plain, Size: 326 bytes --]

Предлагаю сделать вот такой переключатель между alsa и pulseaudio 
(смотреть приложенный патч). Сделал по аналогии с переключателем браузера.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>


[-- Attachment #2: 0001-Added-a-switch-between-alsa-and-pulseaudio.patch --]
[-- Type: text/x-patch, Size: 1320 bytes --]

>From cd186de51026af6bd71315854dc61606fd36df3b Mon Sep 17 00:00:00 2001
From: Anton Midyukov <antohami@altlinux.org>
Date: Wed, 10 Jan 2018 22:37:34 +0700
Subject: [PATCH] Added a switch between alsa and pulseaudio

---
 features.in/sound/config.mk | 6 +++---
 pkg.in/lists/sound/alsa     | 2 ++
 2 files changed, 5 insertions(+), 3 deletions(-)
 create mode 100644 pkg.in/lists/sound/alsa

diff --git a/features.in/sound/config.mk b/features.in/sound/config.mk
index 5761963..96f191c 100644
--- a/features.in/sound/config.mk
+++ b/features.in/sound/config.mk
@@ -6,12 +6,12 @@ use/sound:
 	@$(call add_feature)
 	@$(call add,THE_KMODULES,alsa sound)
 	@$(call add,THE_LISTS,sound/base)
+	@$(call add,THE_LISTS,$$(THE_SOUND))
 
 # ALSA only sound (additional utils needed if using pulseaudio)
 use/sound/alsa: use/sound
-	@$(call add,THE_PACKAGES,apulse)
-	@$(call add,THE_PACKAGES,udev-alsa)
+	@$(call set,THE_SOUND,sound/alsa)
 
 # PulseAudio (useful for per-app levels, dynamic devices and networked sound)
 use/sound/pulse: use/sound
-	@$(call add,THE_LISTS,sound/pulseaudio)
+	@$(call set,THE_SOUND,sound/pulseaudio)
diff --git a/pkg.in/lists/sound/alsa b/pkg.in/lists/sound/alsa
new file mode 100644
index 0000000..dc3226f
--- /dev/null
+++ b/pkg.in/lists/sound/alsa
@@ -0,0 +1,2 @@
+apulse
+udev-alsa
-- 
2.10.4

Re: [devel-distro] Переключатель между alsa и pulseaudio

[Michael Shigorin]

[-- Attachment #1: Type: text/plain, Size: 1104 bytes --]

On Wed, Jan 10, 2018 at 10:45:34PM +0700, Антон Мидюков wrote:
> Предлагаю сделать вот такой переключатель между alsa
> и pulseaudio (смотреть приложенный патч).

Была ещё такая задумка, чтоб при подключении +sound получить
сразу звук на альсе (которая всяко по умолчанию как базовое API),
но с плюшечками вроде udev-alsa; а вот при явном +pulse
подключать егойный список пакетов.

Подумал сейчас и понял, почему она словами толком не пояснялась:
уж больно громоздко.

Пожалуй, сейчас возьму этот патч для начала (только описание
опять-таки надо дополнить целеполаганием, т.е. "зачем так",
ну да поправлю).

> Subject: [PATCH] Added a switch between alsa and pulseaudio

А так всё верно, благодарю.  Можно готовить следующий с +alsa
в тех промежуточных целях, что обсудили, и остающихся конечных
целях образов вроде regular-lxde.

PS: в понедельник пропустил выпуск, к которому вообще-то уже
накопилось патчей (и в регулярки это всё тоже не попало);
надеюсь, на следующей неделе буду внимательней.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

[-- Attachment #2: 0001-sound-added-udev-alsa-in-feature-use-sound-alsa-alsa.patch --]
[-- Type: text/x-patch, Size: 846 bytes --]

>From c13ec2d67b7388ef11586f4ed141c7d19b057425 Mon Sep 17 00:00:00 2001
From: Anton Midyukov <antohami@altlinux.org>
Date: Sat, 6 Jan 2018 16:06:55 +0700
Subject: [PATCH 1/2] sound: added udev-alsa in feature use/sound/alsa (+alsa)

This one should care for default mixer levels upon boot.
---
 features.in/sound/config.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/features.in/sound/config.mk b/features.in/sound/config.mk
index 02b603e..5761963 100644
--- a/features.in/sound/config.mk
+++ b/features.in/sound/config.mk
@@ -10,6 +10,7 @@ use/sound:
 # ALSA only sound (additional utils needed if using pulseaudio)
 use/sound/alsa: use/sound
 	@$(call add,THE_PACKAGES,apulse)
+	@$(call add,THE_PACKAGES,udev-alsa)
 
 # PulseAudio (useful for per-app levels, dynamic devices and networked sound)
 use/sound/pulse: use/sound
-- 
2.10.4


[-- Attachment #3: 0002-sound-added-a-switch-between-alsa-and-pulseaudio.patch --]
[-- Type: text/x-patch, Size: 1581 bytes --]

>From 530d4c1740aa2a5cd03d07c9aa3370a592483068 Mon Sep 17 00:00:00 2001
From: Anton Midyukov <antohami@altlinux.org>
Date: Wed, 10 Jan 2018 22:37:34 +0700
Subject: [PATCH 2/2] sound: added a switch between alsa and pulseaudio

Those images who are fine with ALSA might still benefit
from addons like udev-alsa to have mixer levels restored
on boot; and PulseAudio-bearing images can be based upon
the same intermediate targets now without receiving the
addons they don't need.
---
 features.in/sound/config.mk | 6 +++---
 pkg.in/lists/sound/alsa     | 2 ++
 2 files changed, 5 insertions(+), 3 deletions(-)
 create mode 100644 pkg.in/lists/sound/alsa

diff --git a/features.in/sound/config.mk b/features.in/sound/config.mk
index 5761963..96f191c 100644
--- a/features.in/sound/config.mk
+++ b/features.in/sound/config.mk
@@ -6,12 +6,12 @@ use/sound:
 	@$(call add_feature)
 	@$(call add,THE_KMODULES,alsa sound)
 	@$(call add,THE_LISTS,sound/base)
+	@$(call add,THE_LISTS,$$(THE_SOUND))
 
 # ALSA only sound (additional utils needed if using pulseaudio)
 use/sound/alsa: use/sound
-	@$(call add,THE_PACKAGES,apulse)
-	@$(call add,THE_PACKAGES,udev-alsa)
+	@$(call set,THE_SOUND,sound/alsa)
 
 # PulseAudio (useful for per-app levels, dynamic devices and networked sound)
 use/sound/pulse: use/sound
-	@$(call add,THE_LISTS,sound/pulseaudio)
+	@$(call set,THE_SOUND,sound/pulseaudio)
diff --git a/pkg.in/lists/sound/alsa b/pkg.in/lists/sound/alsa
new file mode 100644
index 0000000..dc3226f
--- /dev/null
+++ b/pkg.in/lists/sound/alsa
@@ -0,0 +1,2 @@
+apulse
+udev-alsa
-- 
2.10.4

Re: [devel-distro] Переключатель между alsa и pulseaudio

[Антон Мидюков]

[-- Attachment #1: Type: text/plain, Size: 2555 bytes --]

10.01.2018 23:37, Michael Shigorin пишет:
> On Wed, Jan 10, 2018 at 10:45:34PM +0700, Антон Мидюков wrote:
>> Предлагаю сделать вот такой переключатель между alsa
>> и pulseaudio (смотреть приложенный патч).
> Была ещё такая задумка, чтоб при подключении +sound получить
> сразу звук на альсе (которая всяко по умолчанию как базовое API),
> но с плюшечками вроде udev-alsa; а вот при явном +pulse
> подключать егойный список пакетов.
>
> Подумал сейчас и понял, почему она словами толком не пояснялась:
> уж больно громоздко.
>
> Пожалуй, сейчас возьму этот патч для начала (только описание
> опять-таки надо дополнить целеполаганием, т.е. "зачем так",
> ну да поправлю).
>
>> Subject: [PATCH] Added a switch between alsa and pulseaudio
> А так всё верно, благодарю.  Можно готовить следующий с +alsa
> в тех промежуточных целях, что обсудили, и остающихся конечных
> целях образов вроде regular-lxde.
Предлагаю добавить +alsa в mixin/regular-desktop вместо /use/sound. 
Разом решим проблемы во всех десктопных регулярках без пульсы с нулевым 
уровнем громкости при перезагрузке системы и невозможностью 
воспроизвести звук в приложениях, умеющих только pulseaudio, при помощи 
apulse.

Остаётся решить вопрос с firefox. Он не может воспроизводить звук без 
пульсы, даже с apulse. Так что нужно определиться, каким регуляркам 
добавить +pulse, а у каких firefox заменить на palemoon. Сейчас +pulse 
только у KDE. Я предлагаю пульсу добавить в сборки gnome3 и cinnamon. 
Соответственно у них оставить firefox, у остальных заменить на palemoon.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>


[-- Attachment #2: 0001-Replace-use-sound-feature-at-alsa-for-desktop-re.patch --]
[-- Type: text/x-patch, Size: 1165 bytes --]

>From 9db8868bccef456860d560396db9e8383c3d77b4 Mon Sep 17 00:00:00 2001
From: Anton Midyukov <antohami@altlinux.org>
Date: Sat, 13 Jan 2018 13:09:53 +0700
Subject: [PATCH] Replace /use/sound feature at +alsa for desktop/regular.
 This will solve the sound problems when using regulars: - The sound when the
 computer is restarted is set to 0 - Some applications do not know how to work
 directly with alsa, apulse allows you to solve this problem partially. Using
 feature +pulse will override use of +alsa feature.

---
 conf.d/mixin.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/conf.d/mixin.mk b/conf.d/mixin.mk
index 40c74da..b5f9589 100644
--- a/conf.d/mixin.mk
+++ b/conf.d/mixin.mk
@@ -19,7 +19,7 @@ mixin/e2k-desktop: use/e2k/x11 use/e2k/sound use/l10n/default/ru_RU \
 
 ### regular.mk
 # common WM live/installer bits
-mixin/regular-desktop: use/x11/xorg use/sound use/xdg-user-dirs
+mixin/regular-desktop: use/x11/xorg +alsa use/xdg-user-dirs
 	@$(call add,THE_PACKAGES,pam-limits-desktop)
 	@$(call add,THE_PACKAGES,installer-feature-desktop-other-fs-stage2)
 	@$(call add,THE_PACKAGES,alterator-notes dvd+rw-tools)
-- 
2.10.4

Re: [devel-distro] Переключатель между alsa и pulseaudio

[Michael Shigorin]

On Sat, Jan 13, 2018 at 04:56:58PM +0700, Антон Мидюков wrote:
> Предлагаю добавить +alsa в mixin/regular-desktop вместо /use/sound. 

Принято.

> Остаётся решить вопрос с firefox. Он не может воспроизводить звук без 
> пульсы, даже с apulse. Так что нужно определиться, каким регуляркам 
> добавить +pulse, а у каких firefox заменить на palemoon. Сейчас +pulse 
> только у KDE. Я предлагаю пульсу добавить в сборки gnome3 и cinnamon. 
> Соответственно у них оставить firefox, у остальных заменить на palemoon.

Мне всё-таки firefox по умолчанию кажется более разумным --
возможно, стоит предложить патч скрипта запуска, при отсутствии
/usr/bin/pulseaudio запускающий firefox-bin через apulse
(локально объехал правкой того, что запускает кнопочка в WM).

> >From 9db8868bccef456860d560396db9e8383c3d77b4 Mon Sep 17 00:00:00 2001
> From: Anton Midyukov <antohami@altlinux.org>
> Date: Sat, 13 Jan 2018 13:09:53 +0700
> Subject: [PATCH] Replace /use/sound feature at +alsa for desktop/regular.
>  This will solve the sound problems when using regulars: - The sound when the
>  computer is restarted is set to 0 - Some applications do not know how to work
>  directly with alsa, apulse allows you to solve this problem partially. Using
>  feature +pulse will override use of +alsa feature.

Поправил, но вообще стоит при написании commit message оставлять
пустую строку между первой (которая суть однострочное описание)
и последующими, чтоб не склеивалось таким макаром.

Ну и объект исправления хорошо бы указывать, как в других коммитах.

Вышло так:

---
mixin.mk: replace use/sound with +alsa for desktop/regular
  
This will solve the sound problems when using regulars:
- The sound when the computer is restarted is set to 0
- Some applications do not know how to work directly with ALSA,
  apulse allows you to solve this problem partially.

Using feature +pulse will override use of +alsa feature.
---

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [devel-distro] Переключатель между alsa и pulseaudio

[Антон Мидюков]

16.01.2018 19:01, Michael Shigorin пишет:
> On Sat, Jan 13, 2018 at 04:56:58PM +0700, Антон Мидюков wrote:
>> Предлагаю добавить +alsa в mixin/regular-desktop вместо /use/sound.
> Принято.
>
>> Остаётся решить вопрос с firefox. Он не может воспроизводить звук без
>> пульсы, даже с apulse. Так что нужно определиться, каким регуляркам
>> добавить +pulse, а у каких firefox заменить на palemoon. Сейчас +pulse
>> только у KDE. Я предлагаю пульсу добавить в сборки gnome3 и cinnamon.
>> Соответственно у них оставить firefox, у остальных заменить на palemoon.
> Мне всё-таки firefox по умолчанию кажется более разумным --
> возможно, стоит предложить патч скрипта запуска, при отсутствии
> /usr/bin/pulseaudio запускающий firefox-bin через apulse
> (локально объехал правкой того, что запускает кнопочка в WM).

Т.е. через apulse звук в firefox работает нормально? Меня на форуме 
ввели в заблуждение, что не помогает. Сам не проверял. Раз так, то пусть 
будет огнелис.
>> >From 9db8868bccef456860d560396db9e8383c3d77b4 Mon Sep 17 00:00:00 2001
>> From: Anton Midyukov <antohami@altlinux.org>
>> Date: Sat, 13 Jan 2018 13:09:53 +0700
>> Subject: [PATCH] Replace /use/sound feature at +alsa for desktop/regular.
>>   This will solve the sound problems when using regulars: - The sound when the
>>   computer is restarted is set to 0 - Some applications do not know how to work
>>   directly with alsa, apulse allows you to solve this problem partially. Using
>>   feature +pulse will override use of +alsa feature.
> Поправил, но вообще стоит при написании commit message оставлять
> пустую строку между первой (которая суть однострочное описание)
> и последующими, чтоб не склеивалось таким макаром.
>
> Ну и объект исправления хорошо бы указывать, как в других коммитах.
>
> Вышло так:
>
> ---
> mixin.mk: replace use/sound with +alsa for desktop/regular
>    
> This will solve the sound problems when using regulars:
> - The sound when the computer is restarted is set to 0
> - Some applications do not know how to work directly with ALSA,
>    apulse allows you to solve this problem partially.
>
> Using feature +pulse will override use of +alsa feature.
> ---
>
Спасибо. Не искушён в коммито-делании :-)

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel-distro] Переключатель между alsa и pulseaudio

[Michael Shigorin]

On Tue, Jan 16, 2018 at 07:17:20PM +0700, Антон Мидюков wrote:
> Т.е. через apulse звук в firefox работает нормально?

Как минимум у меня в сизифном firefox-esr (который уже тоже
покалечен апстримом в части bare alsa) -- да, работает.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info