[Bug 621015] Review Request: rubygem-authlogic - A simple ruby authentication solution

bugzilla at redhat.com bugzilla at redhat.com
Tue Oct 12 19:57:25 UTC 2010


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

Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody at fedoraproject.org    |mtasaka at ioa.s.u-tokyo.ac.jp
               Flag|                            |fedora-review+

--- Comment #7 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp> 2010-10-12 15:57:24 EDT ---
Two comments

- I usually think that writing %doc attribute on -doc subpackage is
  redundant because the name of the rpm already shows that the rpm
  is for documentation.

- Maybe "Patch0: remove-unneeded-rubydebug-dep.patch" is no longer
  needed, because now rubygem-ruby-debug is available on F-12/13/14/15.
  (please try to remove Patch0 and adding BR: rubygem(ruby-debug))

Not blockers, approving.
---------------------------------------------------------
    This package (rubygem-authlogic) 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.



More information about the package-review mailing list