Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 928898 - sys-apps/lm-sensors-3.6.0-r1 - prog/.../rrd.c: error: passing argument 2 of rrd_create from incompatible pointer type [-Wincompatible-pointer-types]
Summary: sys-apps/lm-sensors-3.6.0-r1 - prog/.../rrd.c: error: passing argument 2 of r...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard: fixed in 3.6.2
Keywords: PATCH, PullRequest
Depends on:
Blocks: c99-porting
  Show dependency tree
 
Reported: 2024-04-07 16:34 UTC by Toralf Förster
Modified: 2024-11-01 15:39 UTC (History)
6 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge-info.txt (emerge-info.txt,17.87 KB, text/plain)
2024-04-07 16:34 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,19.89 KB, text/plain)
2024-04-07 16:34 UTC, Toralf Förster
Details
environment (environment,106.50 KB, text/plain)
2024-04-07 16:34 UTC, Toralf Förster
Details
etc.clang.tar.xz (etc.clang.tar.xz,1.14 KB, application/x-xz)
2024-04-07 16:34 UTC, Toralf Förster
Details
etc.portage.tar.xz (etc.portage.tar.xz,24.36 KB, application/x-xz)
2024-04-07 16:34 UTC, Toralf Förster
Details
qlist-info.txt (qlist-info.txt,54.39 KB, text/plain)
2024-04-07 16:34 UTC, Toralf Förster
Details
sys-apps:lm-sensors-3.6.0-r1:20240407-162127.log (sys-apps:lm-sensors-3.6.0-r1:20240407-162127.log,23.64 KB, text/plain)
2024-04-07 16:34 UTC, Toralf Förster
Details
temp.tar.xz (temp.tar.xz,26.02 KB, application/x-xz)
2024-04-07 16:34 UTC, Toralf Förster
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2024-04-07 16:34:05 UTC
too long lines were shrinked:

x86_64-pc-linux-gnu-gcc -DETCDIR="\"/etc\"" -I. -Wall -O2 -Wstrict-prototypes -Wshadow -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-strings -Wnested-externs -Winline -W -Wmissing-prototypes -Wundef -O2 -pipe -march=native -fno-diagnostics-color -c prog/sensord/lib.c -o prog/sensord/lib.ro
x86_64-pc-linux-gnu-gcc -DETCDIR="\"/etc\"" -I. -Wall -O2 -Wstrict-prototypes -Wshadow -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-strings -Wnested-externs -Winline -W -Wmissing-prototypes -Wundef -O2 -pipe -march=native -fno-diagnostics-color -c prog/sensord/rrd.c -o prog/sensord/rrd.ro
prog/sensord/rrd.c: In function 'rrdInit':
prog/sensord/rrd.c:302:40: warning: cast discards 'const' qualifier from pointer target type [-Wcast-qual]
  302 |                 ret = rrd_create(argc, (char**) argv);
      |                                        ^
prog/sensord/rrd.c:302:40: error: passing argument 2 of 'rrd_create' from incompatible pointer type [-Wincompatible-pointer-types]
  302 |                 ret = rrd_create(argc, (char**) argv);
      |                                        ^~~~~~~~~~~~~

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 23.0_systemd-20240406-220503

  -------------------------------------------------------------------

gcc-config -l:
 [1] x86_64-pc-linux-gnu-14 *
clang/llvm (if any):
clang version 18.1.3
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/lib/llvm/18/bin
Configuration file: /etc/clang/x86_64-pc-linux-gnu-clang.cfg
/usr/lib/llvm/18
18.1.3
Python 3.11.8
Available Rust versions:
  [1]   rust-bin-1.77.1 *
php cli (if any):

  HEAD of ::gentoo
commit fb7a347a7c598a29c540f8abca7f612de74ba1ce
Author: Repository mirror & CI <repomirrorci@gentoo.org>
Date:   Sun Apr 7 16:05:56 2024 +0000

    2024-04-07 16:05:56 UTC

emerge -qpvO sys-apps/lm-sensors
[ebuild  N    ] sys-apps/lm-sensors-3.6.0-r1  USE="contrib sensord (-selinux) -static-libs" ABI_X86="(64) -32 (-x32)"
Comment 1 Toralf Förster gentoo-dev 2024-04-07 16:34:06 UTC
Created attachment 889749 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2024-04-07 16:34:07 UTC
Created attachment 889750 [details]
emerge-history.txt
Comment 3 Toralf Förster gentoo-dev 2024-04-07 16:34:08 UTC
Created attachment 889751 [details]
environment
Comment 4 Toralf Förster gentoo-dev 2024-04-07 16:34:09 UTC
Created attachment 889752 [details]
etc.clang.tar.xz
Comment 5 Toralf Förster gentoo-dev 2024-04-07 16:34:10 UTC
Created attachment 889753 [details]
etc.portage.tar.xz
Comment 6 Toralf Förster gentoo-dev 2024-04-07 16:34:11 UTC
Created attachment 889754 [details]
qlist-info.txt
Comment 7 Toralf Förster gentoo-dev 2024-04-07 16:34:12 UTC
Created attachment 889755 [details]
sys-apps:lm-sensors-3.6.0-r1:20240407-162127.log
Comment 8 Toralf Förster gentoo-dev 2024-04-07 16:34:13 UTC
Created attachment 889756 [details]
temp.tar.xz
Comment 9 Maxim P. Dementiev 2024-04-10 19:27:10 UTC
As with changing use flags in https://bugs.gentoo.org/870424#c11 
here "-sensord" helps to bypass this issue.
Comment 10 Jack 2024-05-29 00:01:58 UTC
Upstream PR: https://github.com/lm-sensors/lm-sensors/pull/497
Comment 11 Gabi Falk 2024-05-30 19:19:54 UTC
(In reply to Jack from comment #10)
> Upstream PR: https://github.com/lm-sensors/lm-sensors/pull/497
Brahmajit's PR uses a patch from another upstream PR: https://github.com/lm-sensors/lm-sensors/pull/483 .

As long as there is no response (or any activity at all) upstream to either PR, and version 1.8.0-r4 is the only revision of net-analyzer/rrdtool we have, I believe it's safe to apply either patch.  Personally, I prefer lm-sensors/PR#483 because removing casts was the main idea of the rrdtool API change (see https://github.com/oetiker/rrdtool-1.x/pull/1242), so the best option is to merge Brahmajit's PR: https://github.com/gentoo/gentoo/pull/36352 .
Comment 12 Larry the Git Cow gentoo-dev 2024-10-16 21:33:09 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f56aea21925aea56d034732190345a1deae63b31

commit f56aea21925aea56d034732190345a1deae63b31
Author:     Mike Pagano <mpagano@gentoo.org>
AuthorDate: 2024-10-16 21:32:29 +0000
Commit:     Mike Pagano <mpagano@gentoo.org>
CommitDate: 2024-10-16 21:33:02 +0000

    sys-apps/lm-sensors: Bump lm-sensors and address open bugs, PR
    
    As indicated in bug: 935840, upstream lm-sensors has had no commits
    for three years.
    
    An actively maintained fork by a SUSE kernel developer is located here:
    https://github.com/hramrach/lm-sensors
    
    Change the SRC_URI to this repository.
    
    This also addresses Bug 928898 via upstream commit:
    https://github.com/hramrach/lm-sensors/commit/6b5a19b70867941b3841eb879ebd33e491a67c25
    
    Closes: https://bugs.gentoo.org/928898
    Closes: https://bugs.gentoo.org/935840
    Closes: https://github.com/gentoo/gentoo/pull/36352
    
    Signed-off-by: Mike Pagano <mpagano@gentoo.org>

 sys-apps/lm-sensors/Manifest                |   1 +
 sys-apps/lm-sensors/lm-sensors-3.6.2.ebuild | 223 ++++++++++++++++++++++++++++
 2 files changed, 224 insertions(+)