[Bug 621017] Review Request: rubygem-scruffy - A powerful, clean graphing library for Ruby

[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=621017

Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mtasaka@xxxxxxxxxxxxxxxxxxx

--- Comment #1 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2010-08-05 15:41:33 EDT ---
Some notes:

* Unused macros
  - Defined %ruby_sitelib macro seems used nowhere

* License
-------------------------------------------------------------
CC-BY
./website/javascripts/lightbox.js

LGPLv2+
./website/javascripts/rounded_corners_lite.inc.js
-------------------------------------------------------------
  - License tag should be "MIT and CC-BY and LGPLv2+"
    (also see below)

* Requires
  - "R: ruby(abi) = 1.8" is a must item.
  - "rubygem(hoe)" is for development dependency (for
    Rakefile) and is not needed for runtime
    (Also "hoe" is marked as "s.add_development_dependency"
    in installed .gemspec file)
  - lib/scruffy.rb contains:
------------------------------------------------------------
    17  require 'rubygems'
    18  gem 'builder', '>= 2.0'
    19  require 'builder'
------------------------------------------------------------
    So this package should have "Requires: rubygem(builder)".

* Documents
  - I guess the following files / directories can be
    moved to -doc subpackage (after creating -doc subpackage).
    Would you consider this?
----------------------------------------------------------
%{geminstdir}/Rakefile
%{geminstdir}/config
%{geminstdir}/script
%{geminstdir}/spec
%{geminstdir}/test
%{geminstdir}/website
%{geminstdir}/tasks
%{geminstdir}/setup.rb
%{gemdir}/doc/%{gemname}-%{version}
----------------------------------------------------------

* %check
  - As this gem contains spec/ and test/ directories, please
    add %check section and execute some test programs
    ($ rake test and $ rake spec) there.

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