Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: beep - Beep the PC speaker any number of ways
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Summary: Review Request: beep - Beep the PC speaker any number of ways Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: rhbugs@n-dimensional.de QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
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.... Description:
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: http://ndim.fedorapeople.org/packages/beep/1.2.2-3.fc9/rpmlint.txt
Shipped files: http://ndim.fedorapeople.org/packages/beep/1.2.2-3.fc9/filelist.txt
Build log: http://ndim.fedorapeople.org/packages/beep/1.2.2-3.fc9/build.log
Successful scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=811677
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Andreas Thienemann andreas@bawue.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |andreas@bawue.net AssignedTo|nobody@fedoraproject.org |andreas@bawue.net
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Andreas Thienemann andreas@bawue.net changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|andreas@bawue.net |fabian@bernewireless.net
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|fabian@bernewireless.net |nobody@fedoraproject.org
--- Comment #1 from Fabian Affolter fabian@bernewireless.net 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Jeroen van Meeuwen kanarip@kanarip.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kanarip@kanarip.com AssignedTo|nobody@fedoraproject.org |kanarip@kanarip.com
--- Comment #2 from Jeroen van Meeuwen kanarip@kanarip.com 2008-09-07 10:14:46 EDT --- Good review Fabian, thanks.
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
--- Comment #3 from Hans Ulrich Niedermann rhbugs@n-dimensional.de 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
--- Comment #4 from Kevin Fenzi kevin@tummy.com 2008-09-08 00:13:19 EDT --- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=461404
Hans Ulrich Niedermann rhbugs@n-dimensional.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution| |NEXTRELEASE
--- Comment #5 from Hans Ulrich Niedermann rhbugs@n-dimensional.de 2008-09-08 04:15:11 EDT --- Thanks for reviewing and CVS. Updates are in testing.
package-review@lists.fedoraproject.org