All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] alsactl: Remove standard output definition in systemd unit
@ 2016-12-27 22:47 Paul Menzel
  2016-12-28 16:09 ` Takashi Iwai
  0 siblings, 1 reply; 2+ messages in thread
From: Paul Menzel @ 2016-12-27 22:47 UTC (permalink / raw)
  To: alsa-devel; +Cc: Jordi Mallach


[-- Attachment #1.1: Type: text/plain, Size: 1305 bytes --]

Date: Tue, 8 Jul 2014 07:23:06 +0000

`/lib/systemd/system/alsa-restore.service` specifies
`StandardOutput=syslog`. This overrides the `DefaultStandardOutput`
setting from `/etc/systemd/system.conf`, which the system administrator
can use to specify how output gets logged. In particular, the sysadmin
may want output to go to the journal, or to syslog, or nowhere at all [1].

This patch removes the definition entirely, so the units can use the
system default.

Upstream the patch from the Debian package [2].

[1] https://bugs.debian.org/741123
    "systemd services should not use StandardOutput=syslog; should rely
     on DefaultStandardOutput"
[2] https://sources.debian.net/src/alsa-utils/1.1.2-1/debian/patches/systemd_standardoutput.patch/

Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
CC: Jordi Mallach <jordi@debian.org>
---
 alsactl/alsa-restore.service.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/alsactl/alsa-restore.service.in b/alsactl/alsa-restore.service.in
index f3cd86b..c092c2f 100644
--- a/alsactl/alsa-restore.service.in
+++ b/alsactl/alsa-restore.service.in
@@ -15,4 +15,3 @@ Type=oneshot
 RemainAfterExit=true
 ExecStart=-@sbindir@/alsactl restore
 ExecStop=-@sbindir@/alsactl store
-StandardOutput=syslog
-- 
2.11.0

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] alsactl: Remove standard output definition in systemd unit
  2016-12-27 22:47 [PATCH] alsactl: Remove standard output definition in systemd unit Paul Menzel
@ 2016-12-28 16:09 ` Takashi Iwai
  0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2016-12-28 16:09 UTC (permalink / raw)
  To: Paul Menzel; +Cc: alsa-devel, Jordi Mallach

On Tue, 27 Dec 2016 23:47:32 +0100,
Paul Menzel wrote:
> 
> Date: Tue, 8 Jul 2014 07:23:06 +0000
> 
> `/lib/systemd/system/alsa-restore.service` specifies
> `StandardOutput=syslog`. This overrides the `DefaultStandardOutput`
> setting from `/etc/systemd/system.conf`, which the system administrator
> can use to specify how output gets logged. In particular, the sysadmin
> may want output to go to the journal, or to syslog, or nowhere at all [1].
> 
> This patch removes the definition entirely, so the units can use the
> system default.
> 
> Upstream the patch from the Debian package [2].
> 
> [1] https://bugs.debian.org/741123
>     "systemd services should not use StandardOutput=syslog; should rely
>      on DefaultStandardOutput"
> [2] https://sources.debian.net/src/alsa-utils/1.1.2-1/debian/patches/systemd_standardoutput.patch/
> 
> Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
> CC: Jordi Mallach <jordi@debian.org>

Applied, thanks.


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-12-28 16:09 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-27 22:47 [PATCH] alsactl: Remove standard output definition in systemd unit Paul Menzel
2016-12-28 16:09 ` Takashi Iwai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.