* [ISSUE] lm_sensors service missing
@ 2020-12-29 9:53 Mek101
2021-01-02 0:29 ` ericonr
` (7 more replies)
0 siblings, 8 replies; 9+ messages in thread
From: Mek101 @ 2020-12-29 9:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 989 bytes --]
New issue by Mek101 on void-packages repository
https://github.com/void-linux/void-packages/issues/27525
Description:
<!-- Don't request update of package. We have a script for that. https://alpha.de.repo.voidlinux.org/void-updates/void-updates.txt . However, a quality pull request may help. -->
### System
* xuname:
Void 5.9.16_1 x86_64-musl AuthenticAMD uptodate rFFFF
* package:
lm_sensors-3.6.0_1
### Expected behavior
At the end of the `sensors` setup via `sensors-detect`, the latter prints to enable the `lm_sensors` service to load the appropriate modules at boot time.
### Actual behavior
The output is:
```
Copy prog/init/lm_sensors.init to /etc/init.d/lm_sensors for initialization at boot time.
You should now start the lm_sensors service to load the required kernel modules.
```
Not only the paths are wrong, but there also no sensors service under `/etc/sv/` to start
### Steps to reproduce the behavior
Run the `sensors-detect` command.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors service missing
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
@ 2021-01-02 0:29 ` ericonr
2021-01-02 0:30 ` ericonr
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-02 0:29 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 686 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-753409324
Comment:
The systemd unit is
```
[Unit]
Description=Initialize hardware monitoring sensors
[Service]
EnvironmentFile=/etc/sysconfig/lm_sensors
Type=oneshot
RemainAfterExit=yes
ExecStart=-/sbin/modprobe -qab $BUS_MODULES $HWMON_MODULES
ExecStart=/usr/bin/sensors -s
ExecStop=-/sbin/modprobe -qabr $BUS_MODULES $HWMON_MODULES
[Install]
WantedBy=multi-user.target
```
Looks quite feasible to port to a run script, if you want to try it!
It also has `fancontrol.service` and `sensord.service`, but I think we don't build `sensord`.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors service missing
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
2021-01-02 0:29 ` ericonr
@ 2021-01-02 0:30 ` ericonr
2021-01-02 0:31 ` ericonr
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-02 0:30 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 430 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-753409509
Comment:
Argh, no :/
It's a oneshot... So it would be best to install the `lm_sensors.init` script and tell people to call it in `/etc/rc.local`, assuming that works. Afaik we don't any new oneshot services, since they require using `exec pause` at the end, which is quite the ugly workaround.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors service missing
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
2021-01-02 0:29 ` ericonr
2021-01-02 0:30 ` ericonr
@ 2021-01-02 0:31 ` ericonr
2021-02-22 11:37 ` lm_sensors should include a service Mek101
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-01-02 0:31 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 435 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-753409509
Comment:
Argh, no :/
It's a oneshot... So it would be best to install the `lm_sensors.init` script and tell people to call it in `/etc/rc.local`, assuming that works. Afaik we don't want any new oneshot services, since they require using `exec pause` at the end, which is quite the ugly workaround.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors should include a service
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
` (2 preceding siblings ...)
2021-01-02 0:31 ` ericonr
@ 2021-02-22 11:37 ` Mek101
2021-02-22 15:58 ` ericonr
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: Mek101 @ 2021-02-22 11:37 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 232 bytes --]
New comment by Mek101 on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-783312244
Comment:
What if we put them in [`/etc/modules-load.d/`](https://man.voidlinux.org/modules-load.8)?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors should include a service
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
` (3 preceding siblings ...)
2021-02-22 11:37 ` lm_sensors should include a service Mek101
@ 2021-02-22 15:58 ` ericonr
2021-02-22 16:48 ` Mek101
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-02-22 15:58 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 210 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-783476970
Comment:
Can their utility write a file that works in `/etc/modules-load.d`?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors should include a service
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
` (4 preceding siblings ...)
2021-02-22 15:58 ` ericonr
@ 2021-02-22 16:48 ` Mek101
2021-02-22 16:53 ` ericonr
2021-02-22 17:13 ` Mek101
7 siblings, 0 replies; 9+ messages in thread
From: Mek101 @ 2021-02-22 16:48 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 203 bytes --]
New comment by Mek101 on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-783512259
Comment:
We could either patch it or provide the file with the package
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors should include a service
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
` (5 preceding siblings ...)
2021-02-22 16:48 ` Mek101
@ 2021-02-22 16:53 ` ericonr
2021-02-22 17:13 ` Mek101
7 siblings, 0 replies; 9+ messages in thread
From: ericonr @ 2021-02-22 16:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 266 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-783515495
Comment:
How about try to get it upstream? It should solve the issue for any distro, because a service is really not necessary here.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: lm_sensors should include a service
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
` (6 preceding siblings ...)
2021-02-22 16:53 ` ericonr
@ 2021-02-22 17:13 ` Mek101
7 siblings, 0 replies; 9+ messages in thread
From: Mek101 @ 2021-02-22 17:13 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 219 bytes --]
New comment by Mek101 on void-packages repository
https://github.com/void-linux/void-packages/issues/27525#issuecomment-783529136
Comment:
That would be ideal.
Upstream here: https://github.com/lm-sensors/lm-sensors
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-02-22 17:13 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-29 9:53 [ISSUE] lm_sensors service missing Mek101
2021-01-02 0:29 ` ericonr
2021-01-02 0:30 ` ericonr
2021-01-02 0:31 ` ericonr
2021-02-22 11:37 ` lm_sensors should include a service Mek101
2021-02-22 15:58 ` ericonr
2021-02-22 16:48 ` Mek101
2021-02-22 16:53 ` ericonr
2021-02-22 17:13 ` Mek101
Github messages for voidlinux
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://inbox.vuxu.org/voidlinux-github
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V1 voidlinux-github voidlinux-github/ http://inbox.vuxu.org/voidlinux-github \
voidlinux-github@inbox.vuxu.org
public-inbox-index voidlinux-github
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://inbox.vuxu.org/vuxu.github.voidlinux
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git