[Bug 226381] Merge Review: ruby

bugzilla at redhat.com bugzilla at redhat.com
Sun Jan 20 09:16:04 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: ruby


https://bugzilla.redhat.com/show_bug.cgi?id=226381





------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2008-01-20 04:16 EST -------
Created an attachment (id=292287)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=292287&action=view)
License check list of ruby HEAD

For ruby-1.8.6.111-7.fc9:

* License:
  Please check the license check list attached.
  SUMMARY:
  - From file "COPYING", the license of ruby is "GPLv2 or Ruby",
    not "GPL+ or Ruby".

  - Maybe ruby-rdoc license should be "(GPLv2 or Ruby) and CC-BY"?

  - I guess that mdoc2man.rb (problematic, BSD with advertising)
    is not used anywhere, however would you confirm that?

* Timestamps
  - Please fix the following line to use "cp -p".
--------------------------------------------------------------------------
   314	cp misc/*.el $RPM_BUILD_ROOT%{_datadir}/emacs/site-lisp/ruby-mode
--------------------------------------------------------------------------

* Directory ownership issue
  (Check the list if the result is as you desire).
  - The directory
    %{_libdir}/ruby/%{rubyxver}/%{_normalized_cpu}-%{_target_os}
    should be owned by also non-64-bits archs.

Other thing seems okay.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list