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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]