[Bug 752089] Review Request: rubygem-wirble - Handful of common Irb features, made easy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?

--- Comment #2 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxx> 2011-12-21 11:01:21 EST ---
 License
  - The "main" license should be MIT as COPYING says so.
    setup.rb is written by Minero Aoki and it seems
    this script is used not only by this software
    (by searching with google), so this script is not directly
    this software's product.

    So for now I think the license tag should be "MIT" or
    "MIT and LGPLv2" (not LGPLv2+, setup.rb is version 2.1
    specific).

* Documents
  - The following files should be moved to -doc
---------------------------------------------------
%{geminstdir}/Rakefile
%{geminstdir}/setup.rb
%{geminstdir}/wirble.gemspec
---------------------------------------------------
  - The following files can be marked as %doc.
---------------------------------------------------
%{geminstdir}/_irbrc
---------------------------------------------------

* Macros
  - %ruby_sitelib is just defined but not used anywhere.

Other things seems okay.

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



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]