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

Martin Gieseking <martin.giesek...@uos.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.giesek...@uos.de

--- Comment #10 from Martin Gieseking <martin.giesek...@uos.de> 2011-07-28 
15:21:42 EDT ---
Some more notes:

- I agree, BR ncurses-devel is missing and should be added. It's required to
  get the current terminal size. Without it, default values are used.

- Adapt Source0 according to 
  http://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net

- Choose a more appropriate Group, e.g. Applications/Text

- Add file COPYING to the package.

- Be a bit more specific in %files, i.e. replace 
  %{_mandir}/man1/* with %{_mandir}/man1/%{name}.1*

-- 
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to