Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: GREYCstoration - An image denoising and interpolation tool https://bugzilla.redhat.com/show_bug.cgi?id=310641 ------- Additional Comments From packages@xxxxxxxxxxxxxxxxxx 2007-10-07 15:38 EST ------- 1. You're missing a requires hicolor-icon-theme-0.10-2 for the gui package 2. You're missing a requires tcl for the gui package I'd normally expect a desktop-file-utils buildrequires but it appears OK as it must be being dragged in by another dependency. You're correct about gimp, it is needed. Apart from the issues above I have q query about the gui package, what's the rationale for installing it in the %{_datadir} rather than %{_bindir}? Apart from the above everything seems OK now. For some reason it does not build on x86_64-devel, see http://dribble.org.uk/buildlogs/GREYCstoration-2.5.2-4/8/x86_64/ It appears to be dragging in C code without an extern "C" macro but I have no idea where or why, it compiles fine on F7 and FC-6 for x86_64 and compiles fine in devel for i386. So we have two choices, you can either ask on one of the mailing lists for help from someone who knows a lot more about C++ than me, or (annoyingly!) we might have to excludearch: x86_64. Whatever option you'd rather go with. -- 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, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review