[Bug 310641] Review Request: GREYCstoration - An image denoising and interpolation tool

[bugzilla at redhat.com]

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 at amiga-hardware.com  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.