Bug 461404 - Review Request: beep - Beep the PC speaker any number of ways
Review Request: beep - Beep the PC speaker any number of ways
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeroen van Meeuwen
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2008-09-07 07:09 EDT by Hans Ulrich Niedermann
Modified: 2008-09-08 04:15 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-09-08 04:15:11 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
vanmeeuwen+fedora: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Hans Ulrich Niedermann 2008-09-07 07:09:35 EDT
Spec URL: http://ndim.fedorapeople.org/packages/beep/1.2.2-3.fc9/beep.spec
SRPM URL: http://ndim.fedorapeople.org/packages/beep/1.2.2-3.fc9/beep-1.2.2-3.fc9.src.rpm

Beep allows the user to control the PC speaker with precision,
allowing different sounds to indicate different events. While it
can be run quite happily on the commandline, it's intended place
of residence is within shell/perl scripts, notifying the user when
something interesting occurs. Of course, it has no notion of
what's interesting, but it's real good at that notifying part.

Zero rpmlint output:

Shipped files:

Build log:

Successful scratch build:
Comment 1 Fabian Affolter 2008-09-07 08:59:54 EDT
This is only an informal review.

RPM Lint:  ok
Package name:  ok
Spec file: ok
License:  ok
Actual License: ok
%doc License:  ok
Spec file language:  english
Spec file readable:  ok
Upstream source vs. used tarball:  ok
Compile and Build:
 - F-8: ok
 - F-9: ok
 - rawhide: ok
 - EL-5:  - 

Applicable Package Guidelines:  

Locales:  ok
Shared libs:  n/a

Relocatable:  n/a
Directory and file ownership:  ok
No duplicate files in %files:  ok
File Permissions:  ok
Macro usage:  ok
Code vs. Content:  ok
(Large) Documentation:  ok
%doc affecting runtime:  ok
Header files in -devel package:  ok
Static Libraries in -static package:  ok
pkgconfig Requires:  n/a
Library files:  n/a
Devel requires base package:  n/a
.la libtool archives:  n/a
Duplicate ownership of files/directories:  ok
Remove BuildRoot:  ok
UTF-8 filenames:  ok
Comment 2 Jeroen van Meeuwen 2008-09-07 10:14:46 EDT
Good review Fabian, thanks.

Comment 3 Hans Ulrich Niedermann 2008-09-07 10:23:32 EDT
New Package CVS Request
Package Name: beep
Short Description: Beep the PC speaker any number of ways
Owners: ndim
Branches: F-8 F-9
Comment 4 Kevin Fenzi 2008-09-08 00:13:19 EDT
cvs done.
Comment 5 Hans Ulrich Niedermann 2008-09-08 04:15:11 EDT
Thanks for reviewing and CVS. Updates are in testing.

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