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=504709 --- Comment #5 from Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> 2009-06-15 13:26:48 EDT --- (In reply to comment #4) > > (In reply to comment #2) > > > ? The COPYING file you are packaging claims LGPL as the license. > > > The source files say "You may redistribute it and/or modify it under the > > > same license terms as Ruby." > > > Meanwhile the license tag says "GPLv2 or Ruby" > > > What's going on? :) > > - Actually from the source code this is licensed under the same > > license as ruby. Note that ruby is licensed under "GPLv2 or Ruby" > > (well, a little complicated...), so the license tag should > > be so. > > > > Okay, but the COPYING file that is being packaged says LGPL, which is wrong > according to what you say. Shouldn't we drop the wrong COPYING file? - Removed. > Thanks. Now the package fails to build on F-11 > + rake --trace makemo > /var/tmp/rpm-tmp.qYI3ky: line 41: rake: command not found - Ah.. fixed. > I moved rubygem(rake) from BR(check) to regular BR, but then I got > + rake --trace makemo > /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31: command not > found: allison --path > > I couldn't figure out what package provides allison on F-11. - allison is not packaged on Fedora yet (and I have not checked the dependency...) > Do you know by any chance when these changes in the guidelines will be > approved? I can't find the relevant discussion. Maybe I'm looking at the wrong > mailing lists. - Well, I will re-propose this later on fedora-packaging (after I write a draft: formal proposal needs a draft). http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord-2.0.4-4.fc.src.rpm http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord.spec * Tue Jun 16 2009 Mamoru Tasaka <mtasaka@xxxxxxxxxxxxxxxxxxx> - 2.0.4-4 - Remove COPYING for now - Change BR: now rubygem(rake) is needed to create gettext mo -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review