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

Mamoru Tasaka <mtas...@fedoraproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nob...@fedoraproject.org    |mtas...@fedoraproject.org
               Flag|                            |fedora-review+

--- Comment #7 from Mamoru Tasaka <mtas...@fedoraproject.org> 2011-11-28 
23:42:00 EST ---
Well,

* Description
  - I think the %description section can be expanded to be
    more illustrative, as in "README" file or on the URL.

* Documents
  - I think "README" should be in the main rpm, because it
    says "README".

  - Maybe "CHANGES" can also in the main package

* "gem" command usage
  - I prefert to use "gem -V" (verbose mode), however not a blocker

Apart from these, I see no blocker. (srpm, binary) rpms are clean and can be
correctly installed, and seems to be working.

-----------------------------------------------------------------
    This package (rubygem-rack-cache) is APPROVED by mtasaka
-----------------------------------------------------------------

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