[Bug 562993] Review Request: rubygem-yard - Documentation tool for consistent and usable documentation in 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=562993

Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> changed:

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

--- Comment #1 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2010-02-12 13:26:58 EST ---
Some notes:

* Macros
  - Defined %ruby_sitelib is used nowhere
  - Please use defined %geminstdir also in %files 

* ruby(abi) dependency
  - "Requires: ruby(abi) = 1.8" dependency is missing
    https://fedoraproject.org/wiki/Packaging/Ruby#Ruby_Packaging_Guidelines

* License
--------------------------------------------------------
Overall: MIT

GPLv2 or Ruby
./lib/yard/parser/c_parser.rb
--------------------------------------------------------
  - License tag should be "MIT and (GPLv2 or Ruby)".

* SourceURL
  - We usually use http://gems.rubyforge.org/gems/%{gemname}-%{version}.gem

* Documents
  - "ChangeLog", "LICENSE", "README.md" under %geminstdir should properly be
    marked as %doc (and we usually also mark Rakefile as %doc)
  - Also benckmarks/ docs/ spec/ templates/ directories can be marked
    as %doc
    (and please consider to split these files/directories into -doc
     subpackage)

* %check
  - As this package contains spec/ directory, please add %check section
    and execute some test program (like $ 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]