Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 172428 - Review Request: acpitool - A command line ACPI client for Linux
Summary: Review Request: acpitool - A command line ACPI client for Linux
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: John Mahowald
QA Contact: Fedora Package Reviews List
URL: http://freeunix.dyndns.org:8088/site2...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-04 10:37 UTC by Patrice Dumas
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-11-11 20:40:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Patrice Dumas 2005-11-04 10:37:16 UTC
SRPM Name or Url: fttp://www.environnement.ens.fr/perso/dumas/fc-srpms/acpitool-0.3.0-2.src.rpm

Description: 

AcpiTool is a Linux ACPI client. It's a small command line application,
intended to be a replacement for the apm tool. The primary target
audience are laptop users, since these people are most interested in
things like battery status, thermal status and the ability to suspend
(sleep mode). The program simply accesses the /proc/acpi or /sysfs entries
to get/set acpi values. Hence, a computer running a Linux kernel from
the 2.4.x or 2.6.x series with ACPI enabled is needed.

Comment 1 Patrice Dumas 2005-11-05 23:30:50 UTC
The correct url is in fact:

http://www.environnement.ens.fr/perso/dumas/fc-srpms/acpitool-0.3.0-2.src.rpm

Comment 2 John Mahowald 2005-11-10 17:18:23 UTC
`-mcpu=' is deprecated. Use `-mtune=' or '-march=' instead.

Specify CXXFLAGS to configure, like %configure CXXFLAGS=%{_optflags}

Good:
- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on FC4 i386
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

Have not tested yet.

Comment 3 Patrice Dumas 2005-11-10 23:39:10 UTC
%configure allready defines CXXFLAGS to the right value; the issue is in the
upstream configure.in script where compiler flags are setting although they
shouldn't. I have patched the configure script, now it uses the optflags. See
the srpm here:

http://www.environnement.ens.fr/perso/dumas/fc-srpms/acpitool-0.3.0-3.src.rpm

I also have contacted the upstream.

Comment 4 John Mahowald 2005-11-11 02:26:04 UTC
Clean build now. APPROVED

Comment 5 Christian Iseli 2006-10-18 12:50:59 UTC
Normalize summary field for easy parsing


Note You need to log in before you can comment on or make changes to this bug.