[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 #5 from D Haley <mycae@xxxxxxxxx>  2009-01-28 03:58:00 EDT ---
SPEC URL: http://dhd.selfip.com/427e/mathgl-3.spec
SRPM URL: http://dhd.selfip.com/427e/mathgl-1.8-3.fc10.src.rpm


RPMLint:
$ rpmlint ../SRPMS/mathgl-1.8-3.fc10.src.rpm
../RPMS/i386/mathgl-1.8-3.fc10.i386.rpm mathgl.spec 
mathgl.i386: W: shared-lib-calls-exit /usr/lib/libmgl-glut.so.5.0.0
exit@xxxxxxxxx
2 packages and 1 specfiles checked; 0 errors, 1 warnings.

Koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1088112

>Summary without package name would be nice (as UDAV :-) )
Done.

>Please preserve the time stamps, use ' make install DESTDIR=%{buildroot}
INSTALL="install -p" ' if possible instead of ' make install
INSTALL_ROOT=%{buildroot} '
mathgl *does* use configure, so this is Done.

>Use '%defattr(-,root,root,-)' instead of '%defattr(-, root, root)'
Done.


>Will the upstream fix this issue?

Here is what upstream had to say:
---
I understand badness of usage of exit() function. Unfortunately I
don't know how user can terminate the program by other way using GLUT
window :(. So the only suggestion I have is exclude the module
libmgl-glut from the package. The matter is that this is not the part
of MathGL core and the same functionality can be reached by other
widget classes (FLTK or Qt).
---

So yes, if a suitable solution can be found. I haven't looked into it, it may
be trivial or it may not -- I have not used glut for generating windows.
Without suggestions as to the correct way to modify upstream code, no.

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