[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS

[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=489686





--- Comment #56 from Chitlesh GOORAH <chitlesh at gmail.com>  2009-06-14 07:26:32 EDT ---
(In reply to comment #46)
> 1. The following line:
>   Requires:       libstdc++-devel, atlas-devel, lapack-devel, boost-devel
> should be removed, as the corresponding dependencies are already taken into
> account by the 'BuildRequires:' line, and as the 'Requires:' dependencies are
> automatically calculated:
> https://fedoraproject.org/wiki/Packaging/Guidelines#Requires .

Aren't you confused here ? The Requires is for the -devel package and not for
the main package.


Denis, I would appreciate in the future, if you could just post the lines what
need attention rather a separate SRPM. I nearly mistook your spec for the
review.

The package from my point of view is fine. Any objection before I approve it ?
http://arma.sf.net/fedora/armadillo-0.6.11-7.fc10.src.rpm

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