[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