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=540127 --- Comment #25 from Chen Lei <supercyper@xxxxxxx> 2010-04-24 12:59:17 EDT --- (In reply to comment #24) > * Summary > - "BBS client written in Qt" should be enough Not a blocker > * License > - Double check the licenses of these files and write a just License tag. > src/ssh/crc32.cpp > src/ssh/crc32.h > src/ssh/ssh2.h > src/ssh/ssh1.h > src/wallet.h See http://blog.gmane.org/gmane.linux.redhat.fedora.legal/month=20080901 > * BuildRoot tag should be removed. > * %description > - The first paragraph should be better like this: > QTerm is a BBS client for X Window System. It supports Telnet, SSH1 and SSH2 > protocols, and also supports ZMODEM, URL detection and mouse gesture. > - The second paragraph seems redundant and ambiguious. > * %prep > - The sed commands are not necessary now. The upstream files are all in Unix Not a blocker too. -- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review