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=562504 --- Comment #28 from Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> 2010-02-25 13:14:16 EST --- (In reply to comment #27) > Given all of the additional discussion, I feel I should ask if this is ready > for CVS or if it needs more review. There are currently two issues: - the permissions might be wrong (*.so files) - the tests can't run in the buildsystem I'll check that in the buildsystem, once build and adjust the spec, if nessesary. I'll try to adjust the tests a bit and to run them in the future, but this is not really related to a review issue. Tests should be run, but in this case it's not that easily possible... (The discussion about the tests should be maybe discussed somewhere else, maybe that's why it looks like needing more review.) So, I'd say 'ready for CVS'. -- 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