[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/5] gnu: Add tlp service.
From: |
Mathieu Othacehe |
Subject: |
Re: [PATCH 3/5] gnu: Add tlp service. |
Date: |
Thu, 16 Mar 2017 22:05:30 +0100 |
User-agent: |
mu4e 0.9.18; emacs 25.2.1 |
> I have not been able to see anything related to tlp in
> /var/log/messages. And if I set trace mode (TLP_DEBUG, in /etc/tlp), I
> have an error message: "logger: unknown facility name: debug", which I
> think could be patched in tlp source, maybe by removing "-p debug". Or
> maybe it is logger that needs to be patched, I don't know.
>
> Otherwise syslogd would not be needed here. WDYT?
Well when you call logger -p debug, it means with facility = debug and
priority = default one = notice. syslog does only support predefined
facilities like auth, cron, kern ...
The complete list can be found here (section 19.1):
https://www.gnu.org/software/inetutils/manual/html_node/syslogd-invocation.html
The facility debug does not exist and cannot be created.
So logger -p debug makes no sense to me.
However, debug is a priority, so logger -p user.debug or daemon.debug
would be correct.
I could patch those logger calls in guix tlp package but I prefer to
contact tlp maintainer to see if he can fix it upstream.
>
> BTW, could you consider adding TLP_DEBUG to the service?
It is not needed for now as it enables a broken feature !
I will remove syslogd requirement from this service for now.
Thanks for testing this out !
Mathieu
- Re: [PATCH 1/5] gnu: tlp: Read configuration from /etc/tlp., (continued)