Bug 271661 - (radeontool) Review Request: radeontool - Backlight and video output configuration tool for radeon cards
Review Request: radeontool - Backlight and video output configuration tool fo...
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Andreas Bierfert
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 226302 244330
  Show dependency treegraph
 
Reported: 2007-08-31 11:00 EDT by Till Maas
Modified: 2007-11-30 17:12 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-09-13 00:53:39 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
andreas.bierfert: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Till Maas 2007-08-31 11:00:59 EDT
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 12:12:49 EDT
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 04:36:14 EDT
-----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 06:08:42 EDT
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 18:24:05 EDT
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 01:42:07 EDT
(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 00:53:39 EDT
built for devel

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