[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

[bugzilla at redhat.com]

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


Ray Van Dolson <rayvd at bludgeon.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |rayvd at bludgeon.org
         AssignedTo|nobody at fedoraproject.org    |rayvd at bludgeon.org
               Flag|                            |fedora-review?




--- Comment #2 from Ray Van Dolson <rayvd at bludgeon.org>  2009-03-24 03:49:28 EDT ---
Note that correct SRPM URL should be:
  http://ispbrasil.com.br/ytnef/ytnef-2.6-2.fc10.src.rpm

At initial glance, can you change the license to GPLv2+?.  Also, it's a little
cleaner to use the %{__make}, %{__rm} macros if possible, and you might
consider including ChangeLog as part of the documentation.

I'll do a formal review tomorrow.  Going to bed right now. :-)

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