FS#8762 - gkrellm-2.3.0-1 broken due to lm_sensors 3.0.0-1 upgrade

Attached to Project: Community Packages
Opened by Rodolfo Quesada (rquesada) - Monday, 26 November 2007, 02:44 GMT
Last edited by Roman Kyrylych (Romashka) - Wednesday, 09 January 2008, 20:05 GMT
Task Type Bug Report
Category
Status Closed
Assigned To dtw (dibblethewrecker)
Roman Kyrylych (Romashka)
Architecture All
Severity High
Priority Normal
Reported Version 2007.08-2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 4
Private No

Details

Description:
gkrellm fails to start up, due to update in lm-sensors package to which it is linked against.

Additional info:
* package version(s):
gkrellm-2.3.0-1
* config and/or log files etc:
$ gkrellm
gkrellm: error while loading shared libraries: libsensors.so.3: cannot open shared object file: No such file or directory

Steps to reproduce:
Execute gkrellm, it won't find libsensors.so.3 (Currently, lm-sensors 3 installs libsensors.so.4).
This task depends upon

Closed by  Roman Kyrylych (Romashka)
Wednesday, 09 January 2008, 20:05 GMT
Reason for closing:  Fixed
Comment by Roman Kyrylych (Romashka) - Monday, 26 November 2007, 11:49 GMT
I guess other packages are broken too (e.g. sensors-applet).
Comment by João Rodrigues (gothicknight) - Monday, 26 November 2007, 12:11 GMT
I can confirm this, also compiling gkrellm against the new lib gives major errors:

sysdeps/linux.c: In function ‘libsensors_init’:
sysdeps/linux.c:2060: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token
sysdeps/linux.c:2060: error: ‘feature’ undeclared (first use in this function)
sysdeps/linux.c:2060: error: (Each undeclared identifier is reported only once
sysdeps/linux.c:2060: error: for each function it appears in.)
sysdeps/linux.c:2088: warning: passing argument 1 of ‘sensors_get_detected_chips’ from incompatible pointer type
sysdeps/linux.c:2088: error: too few arguments to function ‘sensors_get_detected_chips’

I'd like to point out that a simple ln -s libsensors.so.4 libsensors.so.3 works fine for me, just some problems with negative voltages being over 2 times the value.
Comment by Baeyens (berbae) - Monday, 26 November 2007, 15:17 GMT
The header file /usr/include/sensors/sensors.h from lm_sensors 3.0.0 is very different from the previous one.
So it seems that the linux.c source file from gtkrellm has to be patched to adapt to it.
Before that, the gkrellm can be compiled with :

make LOCALEDIR=/usr/share/locale without-libsensors=yes || return 1

in the PKGBUILD.

Comment by Jan M. (funkyou) - Wednesday, 28 November 2007, 12:15 GMT
Patch is located here:

http://people.atrpms.net/~hdegoede/gkrellm-2.3.0-libsensors4.patch

Comment by Roman Kyrylych (Romashka) - Thursday, 29 November 2007, 14:24 GMT
fixed on i686.
Comment by Roman Kyrylych (Romashka) - Thursday, 29 November 2007, 14:42 GMT
fixed sensors-applet as well.
Comment by João Rodrigues (gothicknight) - Thursday, 29 November 2007, 14:47 GMT
Nice to know!
I've tested the patch posted here by funkyou (lol?) and compiled successfully gkrellm 2.3.0 using arch's pkgbuild + patch.

Cheers
Comment by Roman Kyrylych (Romashka) - Wednesday, 05 December 2007, 17:20 GMT
It seems gkrellm-2.3.1 is broken.
$ gkrellm
gkrellm: error while loading shared libraries: libsensors.so.3: cannot open shared object file: No such file or directory
$ pacman -Ql lm_sensors | grep libsensors.so
lm_sensors /usr/lib/libsensors.so
lm_sensors /usr/lib/libsensors.so.4
lm_sensors /usr/lib/libsensors.so.4.0.0
Is it possible that 2.3.1 was built against old lm_sensors? /me shrugs
Comment by Baeyens (berbae) - Thursday, 06 December 2007, 13:23 GMT
When rebuilt against lm_sensors 3.0.0, it works.
And the patch is not necessary, it was incorporated to the 2.3.1 source.
I think the new gkrellm 2.3.1 was inadvertently built against the old lm_sensors.
It needs only a rebuilding.
Comment by dtw (dibblethewrecker) - Thursday, 06 December 2007, 14:26 GMT
Well, I guess that's my bad. I thought I did a complete update before I rebuilt it. Will fix it later.
Comment by Börje Holmberg (linfan) - Thursday, 06 December 2007, 18:28 GMT
I can confirm what Baeyens stated. I compiled it myself and it worked without the patch.
Comment by dtw (dibblethewrecker) - Thursday, 06 December 2007, 18:55 GMT
Problem solved..I had lm_sensors-svn....shows when I last used my system!

All updated and should be sorted now.
Comment by Corrado Primier (bardo) - Wednesday, 09 January 2008, 19:55 GMT
Here everything's fine, and it seems to be for everyone else too, since there hasn't been anyone complaining lately. Can this BR be closed?

Loading...