[Bug 510839] Review Request: qrupdate - A Fortran library for fast updates of QR and Cholesky decompositions
bugzilla at redhat.com
bugzilla at redhat.com
Sat Jul 11 15:29:17 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=510839
Jussi Lehtola <jussi.lehtola at iki.fi> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
--- Comment #3 from Jussi Lehtola <jussi.lehtola at iki.fi> 2009-07-11 11:29:16 EDT ---
Thanks for the speedy review!
(In reply to comment #2)
> qrupdate.x86_64: W: unused-direct-shlib-dependency /usr/lib64
> /libqrupdate.so.1.0 /lib64/libm.so.6
> qrupdate.x86_64: W: unused-direct-shlib-dependency /usr/lib64
> /libqrupdate.so.1.0 /lib64/libgcc_s.so.1
> The library is linked against other libraries that it does not use. You have
> to install the rpm and then do "rpmlint qrupdate" to see these. I don't think
> these are problematic; it's not as if those libraries won't be linked in for
> some other reason.
Oh, didn't notice that.
> I'm surprised there's nothing like a header, but I guess that's Fortan. You
> just call the functions and hope you get the arguments the right way 'round.
So was I. I think this is a 'feature' of Fortran 77; Fortran 90 has module
files that do this.
I really don't know *why* anyone would still want to write in a language that
lacks e.g. dynamic memory allocation and has an 8 character length limit in
function names and so on. qrupdate seems to have been written in 2008-2009!!!
(Not to mention that modern Fortran compilers often have lacking F77
capabilities..)
**
New Package CVS Request
=======================
Package Name: qrupdate
Short Description: A Fortran library for fast updates of QR and Cholesky
decompositions
Owners: jussilehtola
Branches: EL-5 F-10 F-11
InitialCC:
--
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