Bug 215038

Summary: Review Request: aumix - ncurses based audio mixer
Product: [Fedora] Fedora Reporter: Gabriel Somlo <somlo>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-11-13 20:41:04 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: 163779    

Description Gabriel Somlo 2006-11-10 18:10:19 UTC
Spec URL: http://www.contrib.andrew.cmu.edu/~somlo/aumix.spec
SRPM URL: http://www.contrib.andrew.cmu.edu/~somlo/aumix-2.8-1.src.rpm
Description: An audio mixer with a compact visual layout based on ncurses.

Comment 1 Gabriel Somlo 2006-11-10 18:12:55 UTC
*** Bug 215037 has been marked as a duplicate of this bug. ***

Comment 2 Jason Tibbitts 2006-11-10 18:13:42 UTC
*** Bug 215037 has been marked as a duplicate of this bug. ***

Comment 3 Parag AN(पराग) 2006-11-11 09:45:01 UTC
Nice application will take this for official review.

Comment 4 Parag AN(पराग) 2006-11-11 11:04:13 UTC
unable to post review. I think bugzilla is running slow


Comment 5 Parag AN(पराग) 2006-11-11 11:21:15 UTC
Review:
+ package builds in mock (development i386) for FC6.
+ rpmlint is silent for RPM and SRPM.
+ source files match upstream.
dc3fc7209752207c23e7c94ab886b340  aumix-2.8.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written but NOT properly indented.
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.  License text included in package.
+ %doc is small; no -doc subpackage required.
+ %doc does not affect runtime.
+ COPYING included in %doc.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc files.
+ no -devel subpackage exists
+ no .la files.
+ translations are available
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+Requires: /bin/sh
           libc.so.6
           libc.so.6(GLIBC_2.0)
           libc.so.6(GLIBC_2.1)
           libc.so.6(GLIBC_2.3)
           libc.so.6(GLIBC_2.3.4)
           libc.so.6(GLIBC_2.4)
           libgpm.so.1
           libncurses.so.5
+ no duplicates in %files.
+ file permissions are appropriate.
+ Ncurses based GUI application
APPROVED.


Comment 6 Parag AN(पराग) 2006-11-11 11:59:18 UTC
Don't forget to CLOSE this bug once you imported package in CVS