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@xxxxxxxxx> 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review