[Bug 49] wxsvg-freeworld - C++ library to create, manipulate and render SVG files

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=49


NicolasChauvet <kwizart at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kwizart at gmail.com
             Blocks|2                           |3
         AssignedTo|rpmfusion-package-          |kwizart at gmail.com
                   |review at rpmfusion.org        |




--- Comment #2 from NicolasChauvet <kwizart at gmail.com>  2008-10-16 11:25:15 ---
There is lot of unused-direct-shlib-dependency from the wxsvg library on wxGTK.
I think it should be the same with the binaries as nothing suggest from the
code that it should use all thoses libraries (look at
/usr/bin/calculette-freeworld )
My guess is that (as freetype-freeworld), you should only provide the library.

Nothing suggest that ffmpeg should be required from the -devel. And even that
the devel should be provided.
It is also a really bad idea to rename the headers name (which result as an API
break). Would be better to install headers in /usr/include/wxsvg-freeworld and
then having them in /usr/include/wxsvg-freeworld/wxSVG/svg.h and
/usr/include/wxsvg-freeworld/wxSVGXML/svgxml.h so that can only be overridden 
by the addition of a -I/usr/include/wxsvg-freeworld CFLAGS.

Your export CFLAGS failed to set RPM_OPT_FLAGS. It is better to avoid this kind
of bad hack and set the FFMPEG_CFLAGS accurately (discovered by libavcodec.pc
if not already done).


-- 
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.