* [PR PATCH] htop: enable sensors support
@ 2021-02-21 16:55 the-eater
2021-02-21 18:18 ` ericonr
` (11 more replies)
0 siblings, 12 replies; 13+ messages in thread
From: the-eater @ 2021-02-21 16:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 847 bytes --]
There is a new pull request by the-eater against master on the void-packages repository
https://github.com/the-eater/neo-void-packages feature/htop/temperature
https://github.com/void-linux/void-packages/pull/28959
htop: enable sensors support
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
libsensors won't be a runtime dependency, and will only be loaded if detected
A patch file from https://github.com/void-linux/void-packages/pull/28959.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-feature/htop/temperature-28959.patch --]
[-- Type: text/x-diff, Size: 922 bytes --]
From d0ef080c420ab5461b070f4b9864f28d2730f246 Mon Sep 17 00:00:00 2001
From: eater <=@eater.me>
Date: Sun, 21 Feb 2021 17:54:24 +0100
Subject: [PATCH] htop: enable sensors support
---
srcpkgs/htop/template | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/srcpkgs/htop/template b/srcpkgs/htop/template
index 2693e232c4b..be377849051 100644
--- a/srcpkgs/htop/template
+++ b/srcpkgs/htop/template
@@ -1,11 +1,11 @@
# Template file for 'htop'
pkgname=htop
version=3.0.5
-revision=1
+revision=2
build_style=gnu-configure
-configure_args="--enable-unicode --enable-cgroup --enable-taskstats"
+configure_args="--enable-unicode --enable-cgroup --enable-taskstats --enable-sensors"
hostmakedepends="automake"
-makedepends="ncurses-devel"
+makedepends="ncurses-devel libsensors-devel"
short_desc="Interactive process viewer"
maintainer="Frank Steinborn <steinex@nognu.de>"
license="GPL-2.0-only"
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
@ 2021-02-21 18:18 ` ericonr
2021-02-21 18:34 ` ericonr
` (10 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: ericonr @ 2021-02-21 18:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 160 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782901180
Comment:
Oh neat, good find.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
2021-02-21 18:18 ` ericonr
@ 2021-02-21 18:34 ` ericonr
2021-02-21 19:03 ` BenBE
` (9 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: ericonr @ 2021-02-21 18:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 265 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782903946
Comment:
Ok, `--enable-sensors` is only in git master, this version should be using `--with-sensors`. Or we can wait for an update :)
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
2021-02-21 18:18 ` ericonr
2021-02-21 18:34 ` ericonr
@ 2021-02-21 19:03 ` BenBE
2021-02-21 19:11 ` ericonr
` (8 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: BenBE @ 2021-02-21 19:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 193 bytes --]
New comment by BenBE on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782908068
Comment:
Next release will take a few more $UnitsOfTime … ;-)
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (2 preceding siblings ...)
2021-02-21 19:03 ` BenBE
@ 2021-02-21 19:11 ` ericonr
2021-02-21 19:34 ` the-eater
` (7 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: ericonr @ 2021-02-21 19:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 246 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782909221
Comment:
So use `--with-sensors` and leave a comment that next version will be using `--enable-sensors` instead :)
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (3 preceding siblings ...)
2021-02-21 19:11 ` ericonr
@ 2021-02-21 19:34 ` the-eater
2021-02-21 20:14 ` BenBE
` (6 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: the-eater @ 2021-02-21 19:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 332 bytes --]
New comment by the-eater on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782912620
Comment:
haha, so default value is `check` so it worked™ anyhow.
also
`configure: WARNING: unrecognized options: --enable-cgroup, --enable-taskstats` should I just remove those in that case?
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (4 preceding siblings ...)
2021-02-21 19:34 ` the-eater
@ 2021-02-21 20:14 ` BenBE
2021-02-24 13:03 ` [PR PATCH] [Updated] " the-eater
` (5 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: BenBE @ 2021-02-21 20:14 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 449 bytes --]
New comment by BenBE on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-782918754
Comment:
The options for `configure` have been renamed recently (cf. https://github.com/htop-dev/htop/pull/482) accompanied by changing the defaults to check for the dependencies to be present. If you want to get a stable build (ensuring features are actually compiled in) it's best to specify those options explicitly.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PR PATCH] [Updated] htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (5 preceding siblings ...)
2021-02-21 20:14 ` BenBE
@ 2021-02-24 13:03 ` the-eater
2021-02-24 13:03 ` the-eater
` (4 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: the-eater @ 2021-02-24 13:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 852 bytes --]
There is an updated pull request by the-eater against master on the void-packages repository
https://github.com/the-eater/neo-void-packages feature/htop/temperature
https://github.com/void-linux/void-packages/pull/28959
htop: enable sensors support
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
libsensors won't be a runtime dependency, and will only be loaded if detected
A patch file from https://github.com/void-linux/void-packages/pull/28959.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-feature/htop/temperature-28959.patch --]
[-- Type: text/x-diff, Size: 949 bytes --]
From 2cce3639cbb4bef06bbe5870f5e998c5052e1049 Mon Sep 17 00:00:00 2001
From: eater <=@eater.me>
Date: Sun, 21 Feb 2021 17:54:24 +0100
Subject: [PATCH] htop: enable sensors support
---
srcpkgs/htop/template | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/srcpkgs/htop/template b/srcpkgs/htop/template
index 2693e232c4b..96fd8b5de16 100644
--- a/srcpkgs/htop/template
+++ b/srcpkgs/htop/template
@@ -1,11 +1,12 @@
# Template file for 'htop'
pkgname=htop
version=3.0.5
-revision=1
+revision=2
build_style=gnu-configure
-configure_args="--enable-unicode --enable-cgroup --enable-taskstats"
+# NOTE: change --with-sensors to --enable-sensors next update
+configure_args="--enable-unicode --with-sensors"
hostmakedepends="automake"
-makedepends="ncurses-devel"
+makedepends="ncurses-devel libsensors-devel"
short_desc="Interactive process viewer"
maintainer="Frank Steinborn <steinex@nognu.de>"
license="GPL-2.0-only"
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (6 preceding siblings ...)
2021-02-24 13:03 ` [PR PATCH] [Updated] " the-eater
@ 2021-02-24 13:03 ` the-eater
2021-02-24 14:59 ` [PR PATCH] [Merged]: " ericonr
` (3 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: the-eater @ 2021-02-24 13:03 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 210 bytes --]
New comment by the-eater on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-785059974
Comment:
removed `--enable-cgroup --enable-taskstats` since they don't exist
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PR PATCH] [Merged]: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (7 preceding siblings ...)
2021-02-24 13:03 ` the-eater
@ 2021-02-24 14:59 ` ericonr
2021-05-17 16:07 ` dkwo
` (2 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: ericonr @ 2021-02-24 14:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 671 bytes --]
There's a merged pull request on the void-packages repository
htop: enable sensors support
https://github.com/void-linux/void-packages/pull/28959
Description:
#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
libsensors won't be a runtime dependency, and will only be loaded if detected
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (8 preceding siblings ...)
2021-02-24 14:59 ` [PR PATCH] [Merged]: " ericonr
@ 2021-05-17 16:07 ` dkwo
2021-05-17 18:15 ` the-eater
2021-05-17 18:22 ` dkwo
11 siblings, 0 replies; 13+ messages in thread
From: dkwo @ 2021-05-17 16:07 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 401 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-842448507
Comment:
I do not see any temperature in `Available meters`.
Is any further configuration required?
```
$ htop --version
htop 3.0.5
$ xuname
Void 5.11.21_1 x86_64-musl GenuineIntel uptodate hold rrnFFFF
$ xpkg -v | grep sensors
libsensors-3.6.0_1
```
Thanks.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (9 preceding siblings ...)
2021-05-17 16:07 ` dkwo
@ 2021-05-17 18:15 ` the-eater
2021-05-17 18:22 ` dkwo
11 siblings, 0 replies; 13+ messages in thread
From: the-eater @ 2021-05-17 18:15 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 185 bytes --]
New comment by the-eater on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-842531384
Comment:
Display Options -> Also show CPU Tempature
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: htop: enable sensors support
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
` (10 preceding siblings ...)
2021-05-17 18:15 ` the-eater
@ 2021-05-17 18:22 ` dkwo
11 siblings, 0 replies; 13+ messages in thread
From: dkwo @ 2021-05-17 18:22 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 146 bytes --]
New comment by dkwo on void-packages repository
https://github.com/void-linux/void-packages/pull/28959#issuecomment-842535654
Comment:
Thanks.
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2021-05-17 18:22 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-21 16:55 [PR PATCH] htop: enable sensors support the-eater
2021-02-21 18:18 ` ericonr
2021-02-21 18:34 ` ericonr
2021-02-21 19:03 ` BenBE
2021-02-21 19:11 ` ericonr
2021-02-21 19:34 ` the-eater
2021-02-21 20:14 ` BenBE
2021-02-24 13:03 ` [PR PATCH] [Updated] " the-eater
2021-02-24 13:03 ` the-eater
2021-02-24 14:59 ` [PR PATCH] [Merged]: " ericonr
2021-05-17 16:07 ` dkwo
2021-05-17 18:15 ` the-eater
2021-05-17 18:22 ` dkwo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).