Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 116661 - ksensors 0.7.3 ebuild update suggestion
Summary: ksensors 0.7.3 ebuild update suggestion
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on: 173821
Blocks:
  Show dependency tree
 
Reported: 2005-12-24 21:30 UTC by Anton Bolshakov
Modified: 2007-04-15 02:11 UTC (History)
0 users

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


Attachments
patch to display thermal info provided by ibm_acpi (ksensors-0.7.3-ibm-acpi.patch,6.92 KB, patch)
2005-12-24 21:32 UTC, Anton Bolshakov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anton Bolshakov 2005-12-24 21:30:23 UTC
ksensors is a nice front-end to lm_sensors.
It also displays same /proc/* information such as cpuinfo and cpu themperature.
Since lm_sensors doesn't work with ibm thinkpad laptops and even dangerous to use I want to make this optional(disabled if acpi enabled). I also has extended functionality of ksensors to display /proc/acpi/ibm/thermal.

ksensors-0.7.3-r1.ebuild:
---------------------------------------------
# Copyright 1999-2005 Gentoo Foundation
# Distributed under the terms of the GNU General Public License v2
# $Header: /var/cvsroot/gentoo-x86/kde-misc/ksensors/ksensors-0.7.3.ebuild,v 1.6 2005/10/09 18:02:46 metalgod Exp $

inherit kde

DESCRIPTION="KSensors - a nice lm_sensors frontend for KDE"
SRC_URI="mirror://sourceforge/ksensors/${P}.tar.gz"
HOMEPAGE="http://ksensors.sourceforge.net/"
LICENSE="GPL-2"

SLOT="0"
KEYWORDS="amd64 x86"
IUSE="acpi"

DEPEND="!acpi? sys-apps/lm_sensors"

need-kde 3

src_unpack()
{
        kde_src_unpack
        epatch ${FILESDIR}/${P}-ibm-acpi.patch
        cd ${S}
        rm -f config.cache
}
---------------------------------------
patch attached in the following message.
Comment 1 Anton Bolshakov 2005-12-24 21:32:43 UTC
Created attachment 75480 [details, diff]
patch to display thermal info provided by ibm_acpi

Display more thermal info from: /proc/acpi/ibm/thermal
Comment 2 Stefan Schweizer (RETIRED) gentoo-dev 2006-11-20 12:04:28 UTC
can you please submit this patch upstream. It will find its way back into Gentoo then. Thanks
Comment 3 Carsten Lohrke (RETIRED) gentoo-dev 2007-04-09 17:01:18 UTC
As this application is abandoned, resolution upstream is unreasonable. Using bug 173821 as tracker.
Comment 4 Wulf Krueger (RETIRED) gentoo-dev 2007-04-15 02:11:59 UTC
ksensors needs sys-apps/lm_sensors to build correctly:

In file included from lmsensorsdock.h:34,
                 from main.cpp:24:
lmsensors.h:26:29: error: sensors/sensors.h: No such file or directory

Your patch doesn't change that. It may add the desired functionality (which I can't verify) but the DEPEND in your suggested ebuild is wrong. (And the acpi USE flag can't be used for this anyway.)

I've added an "ibmacpi" local USE flag which triggers your patch to be included.

I've just committed ksensors-0.7.3-r1 to CVS.