Bug 271661 (radeontool)

Summary: Review Request: radeontool - Backlight and video output configuration tool for radeon cards
Product: [Fedora] Fedora Reporter: Till Maas <opensource>
Component: Package ReviewAssignee: Andreas Bierfert <andreas.bierfert>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, pknirsch
Target Milestone: ---Flags: andreas.bierfert: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-09-13 04:53:39 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 226302, 244330    

Description Till Maas 2007-08-31 15:00:59 UTC
Spec URL: http://till.fedorapeople.org/review/radeontool.spec
SRPM URL: http://till.fedorapeople.org/review/radeontool-1.5-1.fc7.src.rpm
Description: Radeontool may switch the backlight and external video output on and off.  Use radeontool at your own risk, it may damage your hardware.

radeontool was previously packaged in pm-utils, now it goes its own way.

Comment 1 Till Maas 2007-09-07 16:12:49 UTC
Spec URL: http://till.fedorapeople.org/review/radeontool.spec
SRPM URL: http://till.fedorapeople.org/review/radeontool-1.5-2.fc7.src.rpm

- add Conflicts with old pm-utils package

Comment 2 Andreas Bierfert 2007-09-08 08:36:14 UTC
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

OK: rpmlint is silent
OK: package naming
OK: spec name
OK: zlib license
OK: license field matches license
OK: spec files English
OK: spec is legible
OK: source matches upstream:
8065eebe5a2b163e43b40461bfe49a56  radeontool-1.5.tar.gz
8065eebe5a2b163e43b40461bfe49a56  rpmbuild/SOURCES/radeontool-1.5.tar.gz
OK: build checked and worked fine on f7/i386
OK: BR are fine
OK: no duplicate files
OK: file permissions are correct
OK: clean section
OK: macro usage
OK: content is ok
OK: %doc does not affect runtime
OK: %install section

The package
2dad2d98dcb108a82a95b9201f80daf9  radeontool-1.5-2.fc7.src.rpm
is hereby approved.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)

iD8DBQFG4l5aQEQyPsWM8csRAialAKC8nogyYGA1BRykwi77+OjjZEfJegCeIpmL
f1xzDptULMJVaWUPz2vLKBs=
=9NQm
-----END PGP SIGNATURE-----

It would be nice that before building it you add the '-p' flag to the install
argument to preserve the timestamps.

Comment 3 Till Maas 2007-09-08 10:08:42 UTC
New Package CVS Request
=======================
Package Name: radeontool
Short Description: Backlight and video output configuration tool for radeon cards
Owners: till,pknirsch
Branches: F-7
Cvsextras Commits: no

If it is possible for you, please also add ajax to watchcommits and commit for
devel.

Comment 4 Kevin Fenzi 2007-09-09 22:24:05 UTC
cvs done. 

The command line tool doesn't have the ability to just do watchcommits and
commit  access. You should be able to use the web interface to setup ajax for
those. 
Let me know if you run into any problems doing so. 

Comment 5 Till Maas 2007-09-10 05:42:07 UTC
(In reply to comment #4)

> The command line tool doesn't have the ability to just do watchcommits and
> commit  access. You should be able to use the web interface to setup ajax for
> those. 
> Let me know if you run into any problems doing so. 

I do not see a way to add him. Maybe he is the only one, who can change it then
by adding himself to the package. I will ask him then, to do this.

Comment 6 Till Maas 2007-09-13 04:53:39 UTC
built for devel