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

Ankur Sinha <sanjay.an...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #11 from Ankur Sinha <sanjay.an...@gmail.com> 2010-05-29 03:08:43 
EDT ---
hey,

[an...@mether SPECS]$ rpmlint radiotray.spec
/var/lib/mock/fedora-rawhide-i386/result/*.rpm
radiotray.spec: W: invalid-url Source0:
http://downloads.sourceforge.net/radiotray/radiotray-0.5.1.tar.gz <urlopen
error timed out>
radiotray.noarch: W: no-manual-page-for-binary radiotray
radiotray.src: W: invalid-url Source0:
http://downloads.sourceforge.net/radiotray/radiotray-0.5.1.tar.gz <urlopen
error timed out>
2 packages and 1 specfiles checked; 0 errors, 3 warnings.

Only one worth looking at is the second one. I'd suggest writing a man page for
the package. 

You could also try to make this work on python3. 

Apart from this, it looks good :)

XXX APPROVED XXX

-- 
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.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to