[Bug 674080] Review Request: rubygem-acts_as_audited - Ruby extension logging all changes to your models in an audits table

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

--- Comment #2 from Bohuslav Kabrda <bkabrda@xxxxxxxxxx> 2011-09-07 06:43:31 EDT ---
Errata:
* The license is really MIT, sorry.

Additional comments:
* The first two lines (definition of %ruby_sitelib and %ruby_sitearch is not
necessarry in this spec file.
* Have you tested the app with Rails 3?
* Consider using %dir %{geminstdir} instead of %{geminstdir} in files - it
solves the problem with files being listed multiple times.
* You should use ruby(rubygems) in Require instead of just rubygems.
* Include BuildRequires: ruby.
* The %defattr(-, root, root, -) in %files section is not needed.
* You should include %check section and run the tests (something like
RUBYOPT="I%{buildroot}%{geminstdir}/lib Itest" testrb test/*_test.rb should
do).
* Reference to koji task:
https://koji.fedoraproject.org/koji/taskinfo?taskID=3332026

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