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=519538 --- Comment #7 from Christian Krause <chkr@xxxxxxxxxxx> 2009-09-16 16:37:15 EDT --- Thank you very much for the review. (In reply to comment #6) > Ping? There are some small changes that need to be made, otherwise the package > is OK. Sorry for the late response. I came just back from 2 weeks of vacation. ;-) The new packages can be found here: Spec URL: http://chkr.fedorapeople.org/review/clutter-sharp.spec SRPM URL: http://chkr.fedorapeople.org/review/clutter-sharp-0-0.4.20090828.fc11.src.rpm (In reply to comment #5) > clutter-sharp-devel.x86_64: W: summary-not-capitalized pkgconfig file for > clutter-sharp > clutter-sharp-devel.x86_64: W: no-documentation > > The first should probably be fixed. The default spec file has > > Development files for %{name} > > which could be used. Esp. since we're packaging more than just the .pc file! Fixed. > OK build dependencies complete (Koji) > > FAIL on the other hand, runtime dependencies are not complete > > rpm -q --requires clutter-sharp > > rpm -e clutter 2>out.txt > cat out.txt | grep sharp > > shows that the dependency on clutter is not picked up; this should be > added manually. Fixed. > Also, once clutter-gtk can be enabled (prod the maintainer for an update?) Unfortunately clutter-gtk is not yet released yet: http://www.clutter-project.org/sources/clutter-gtk/0.10/ . I'll try to keep this in mind and create a bug entry for clutter-gtk once 0.10.3 is released. > you'd want to Requires: clutter-gtk, and meanwhile, now you want to > %exclude %{_libdir}/pkgconfig/clutter-gtk-sharp.pc Fixed. -- 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