[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 William Moreno changed: What|Removed |Added Status|NEW |CLOSED

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 William Moreno changed: What|Removed |Added Flags|

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 --- Comment #10 from Fotios Tsiadimos --- Hi William: I'll fix the issues you mentioned above. And definitely - please review my program and I can also review someone's program informally. Thanks for your help. Fotis

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 William Moreno changed: What|Removed |Added CC|

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 William Moreno changed: What|Removed |Added CC|

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 Fotios Tsiadimos changed: What|Removed |Added Blocks||177841

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 --- Comment #7 from Fotios Tsiadimos ftsiadi...@gmail.com --- Thank you a lot for your help! I removed defattr, %clean-section, rm -rf %{buildroot}. I fixed my GPL license to be GPLv3 and I added -m 644 for my man page In the requires I

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 --- Comment #6 from Jens Lody fed...@jenslody.de --- Informal review: I do not use python myself, so I'm not familiar to it. Did you read: https://fedoraproject.org/wiki/Packaging:Python ? If I understand correctly you should not use the

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 Christopher Meng i...@cicku.me changed: What|Removed |Added CC||i...@cicku.me

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 --- Comment #4 from Jens Lody fed...@jenslody.de --- This is a snippet of fedora-review running on your source-rpm with rawhide as target configuration: Issues: === - Sources used to build the package match the upstream source, as

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 Jens Lody fed...@jenslody.de changed: What|Removed |Added CC||fed...@jenslody.de ---

[Bug 1245255] Review Request: netspy2ban - GUI Networking Tool and Fail2ban Controller

https://bugzilla.redhat.com/show_bug.cgi?id=1245255 --- Comment #5 from Fotios Tsiadimos ftsia...@redhat.com --- Hello Jens, Thank you for your feedback. I replace the link and now this issue is fixed, please let me know if I need to fix something else Source checksums