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: gimp-resynthesizer - Gimp plugin for texture synthesis


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250210


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[EMAIL PROTECTED]    |[EMAIL PROTECTED]




------- Additional Comments From [EMAIL PROTECTED]  2007-08-18 17:41 EST -------
Well,

# Top comment is unnecessary, summary tag is enough.

# license tag need to be fix, as the license policy has been changed, you need
to set explicitly the version of the licensed package now.
So, set it to GPLv2 as your package is.

# Yous should use the recommanded default buildroot which is:
  BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

# RPM variable style VS RPM macros style:
You should use one of them, not mixe them in spec.
Also there are duplicate command is the spec, should be fix.

# Package doesn't honor RPM_OPT_FLAGS

# Failed to build on mock F-devel
-------------------------------------------------------------------------------
[...]
g++ `gimptool --cflags` -O3 -Wall -fno-common -ffast-math -frename-registers
-fomit-frame-pointer -o resynth resynth.cc `gimptool --libs` -lm
resynth.cc:83: error: template with C linkage
resynth.cc:979: error: expected `}' at end of input
make: *** [resynth] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.33961 (%build)
-------------------------------------------------------------------------------



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to