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=600914 --- Comment #2 from Richard Fearn <richardfearn@xxxxxxxxx> 2010-06-08 14:19:40 EDT --- Thanks for the review Thomas! > - %build could be simplified, with a for loop, but ok I don't know why I didn't do that before! That will be in the next release. > - license is unclear: The webside says GPL, the COPYING contains GPLv2. > Is it GPLv2 only, GPLv2+ ? > Please ask upstream to clearify this and make proper headers in a next > release. I have asked. > - %desctiption: > When searching for python or emacs or .... this program will be found, which > is not so good... > What's your opinion towards this? I see your point. I've removed the list of programs now. This means that if support for any new programs is added, the description won't need to be modified. I also fixed a few typos. The current description is: "txt2regex is a regular expression wizard that converts human sentences to regexes. "In a simple interactive interface, you just answer questions and build your own regex for a large variety of software and programming languages." I'll upload a new spec/SRPM when I've heard from upstream. Thanks for your feedback! -- 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