[Bug 478372] Review request: Mathgl - Scientific plotting library.

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


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





--- Comment #10 from D Haley <mycae at yahoo.com>  2009-07-09 09:22:55 EDT ---
SPEC URL: http://dhd.selfip.com/427e/mathgl-1.9-1.spec
SRPM URL: http://dhd.selfip.com/427e/mathgl-1.9-1.fc10.src.rpm


Rpmlint provides one warning worth mentioning:
mathgl.i386: W: unused-direct-shlib-dependency /usr/lib/libmgl.so.5.0.0
/usr/lib/libgomp.so.1
mathgl.i386: W: unused-direct-shlib-dependency /usr/lib/libmgl.so.5.0.0
/usr/lib/libgslcblas.so.0

Using the suggested fix (below) to set libtool to tell the linker to "link as
needed" fails.

sed -i -e 's! -shared ! -Wl,--as-needed\0!g' libtool

Modifying the GSL_FLAGS before the ./configure to "-lgsl" also causes the build
to fail:
/usr/lib/libgsl.so.0: undefined reference to `cblas_ztrsv'
/usr/lib/libgsl.so.0: undefined reference to `cblas_scasum'
/usr/lib/libgsl.so.0: undefined reference to `cblas_izamax'
...

Is rpmlint checking this correctly? The functions that mgl needs are the
"special functions" of GSL, namely Airy and elliptic integral computation. 

I think it is safe to ignore this warning.

>- Drop the explicit Require:
> Requires: gsl 
Done

> - Update to 1.9.0.1, then I'll do the review.  
The 1.9.0.1 tarball appears to be missing the fonts dir (note that the tarball
has halved in size between versions!), and the build fails when make install
cannot find these fonts, which is explicitly asked for the makefile. I have
however updated to 1.9 which does have the font data, and emailed upstream for
advice on what their intent is with respect to this in 1.9.0.1. Did they omit
the fonts inadvertently or are they intending to use system fonts? 

No such discussion is apparent on their site -- according to their news, the
.0.1 release is targeted at fixing a win32 specific bug.

- Keep the time stamps on the converted files with...
Fixed

- Add space after the last entry in %changelog.
Fixed

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