[Bug 497639] Review Request: gmixer - Just a simple audio mixer

bugzilla at redhat.com bugzilla at redhat.com
Thu May 21 11:56:40 UTC 2009


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=497639


leigh scott <leigh123linux at googlemail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?                   |




--- Comment #8 from leigh scott <leigh123linux at googlemail.com>  2009-05-21 07:56:36 EDT ---
(In reply to comment #7)
> This is completely different package. Above full review doesnt count on this
> one. Will make new one after these few errors are fixed. Ok, lets see;
> 
> --------------------------------------------------------------------
> - https://fedoraproject.org/wiki/Packaging/Python#System_Architecture - says if
> you are installing something to global site_packages directory use following:
> %{!?python_sitelib: %global python_sitelib %(%{__python} -c "from
> distutils.sysconfig import get_python_lib; print get_python_lib()")}
> 
> Please change python_sitearch with python_sitelib as stated at link above.
> 
> Dont forget to change all %{python_sitearch} with new one.
> --------------------------------------------------------------------
> 
> --------------------------------------------------------------------
> - remove --vendor.
> --------------------------------------------------------------------
> 
> --------------------------------------------------------------------
> - desktop file has Encoding. It is assumed that encoding is UTF-8 by default so
> that line is useless. Please remove it.
> --------------------------------------------------------------------  

Ok, here's the new spec * SRPM

Spec: http://leigh123linux.fedorapeople.org/review/gmixer/4/gmixer.spec


SRPM:
http://leigh123linux.fedorapeople.org/review/gmixer/4/gmixer-1.3-3.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list