[Bug 695058] Review Request: transgui - An App to remotely control a Transmission Bit-Torrent client

bugzilla at redhat.com bugzilla at redhat.com
Sun Nov 13 06:11:57 UTC 2011


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

--- Comment #20 from Praveen Kumar <kumarpraveen.nitdgp at gmail.com> 2011-11-13 01:11:55 EST ---
(In reply to comment #19)
> Would you like to update the package to the latest upstream release? There's
> also a source tarball available now. 
done
> Just two quick comments:
> - Currently, the debuginfo package is empty. It's probably because of the fpc 
>   flag -g- applied in the Makefiles. You should patch them appropriately.
rpmlint is not throwing any error about empty debuginfo.
> - Also, drop the Encoding entry from the .desktop file as it's deprecated.
done


[daredevil at localhost SPECS]$ rpmlint -i transgui.spec
../SRPMS/transgui-3.2-4.fc16.src.rpm ../RPMS/i686/transgui-3.2-4.fc16.i686.rpm 
transgui.spec: W: invalid-url Source0:
https://transmisson-remote-gui.googlecode.com/files/transgui-3.2-src.zip HTTP
Error 404: Not Found
The value should be a valid, public HTTP, HTTPS, or FTP URL.

transgui.src: W: invalid-url Source0:
https://transmisson-remote-gui.googlecode.com/files/transgui-3.2-src.zip HTTP
Error 404: Not Found
The value should be a valid, public HTTP, HTTPS, or FTP URL.

transgui.i686: W: no-manual-page-for-binary transgui
Each executable in standard binary directories should have a man page.

2 packages and 1 specfiles checked; 0 errors, 3 warnings.

Updated SPEC and SRPM
SPEC: http://kumarpraveen.fedorapeople.org/transgui/transgui.spec
SRPM: http://kumarpraveen.fedorapeople.org/transgui/transgui-3.2-4.fc16.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