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


David Timms <dti...@iinet.net.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dti...@iinet.net.au




--- Comment #3 from David Timms <dti...@iinet.net.au>  2009-01-02 17:05:17 EDT 
---
Really minor suggestions:
- since the spec mostly has a double blank line between sections, you should do
that also between build and install sections.

- The description could be improved (spelling and case at least). Perhaps:
Description:
fspy is an easy to use linux filesystem activity monitoring tool. It is
is small, fast and handles system resources conservatively. Features 
include being able to apply filters, use diffing and set a custom output
format in order to achieve the best results.

- License: Since neither the web site nor readme contains a license statement,
perhaps this could be requested of the upstream (since each source file
includes a GPLv2+ header).

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to