[Bug 481355] Review Request: nurbs++ - A C++ library to manipulate and create NURBS curves and surfaces.

bugzilla at redhat.com bugzilla at redhat.com
Tue Jul 7 18:48:16 UTC 2009


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=481355





--- Comment #14 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp>  2009-07-07 14:48:15 EDT ---
(In reply to comment #13)
> >  - Also configure.in suggests that ImageMagick support
> >    should be enabled by default. Would you try to
> >    add "BR: ImageMagick-devel"?
> Done

- Ah, no... I missed that in fact all ImageMagick related parts
  are commented out in configure.in and currently ncurbs++ does not
  use ImageMagick. You can remove BR: ImageMagick-devel, sorry.

> >- cppunit-devel is available on Fedora. Would you try
> >    to add "BR: cppunit-devel" ?
> 
> I avoided this as unfortunately their cppunit tests do not compile, and I am
> not familiar with that particular framework. I suspect this would requires some
> modest level of patching to fix.

- Okay. It seems that cppunit is used only to build test program.

> >  - $ rpmlint nurbs++ shows lots of rpmlint warnings related to undefined non-weak symbols:
> 
> Odd, my rpmlint (which is up-to-date) does not catch this error. Anyway, its
> clearly was problem, and is now Fixed.

- rpmlint can also be used for "installed" rpm. Then I again
  failed to notice that rpmlint complains also about this warnings
  for both libmatrixI.so and libmatrixN.so....
  Both needs linking against libmatrix.so . Please also this fix.

-- 
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.




More information about the package-review mailing list