[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 #50 from Denis Arnaud <denis.arnaud_fedora at m4x.org>  2009-06-13 17:01:38 EDT ---
The following is an attempt to build with Boost:
-----------------------------------
Spec URL: http://denisarnaud.fedorapeople.org/armadillo/0611/9/armadillo.spec
SRPM URL:
http://denisarnaud.fedorapeople.org/armadillo/0611/9/armadillo-0.6.11-9.fc10.src.rpm
-----------------------------------  

Boost is detected by CMake, but it seems that the Boost libraries are still not
needed (most of Boost is made of just headers). However, as when ARMA_USE_BOOST
is set to true, some Boost headers (e.g., Boost Date-Time) are included, it is
indeed required to add the dependency on boost-devel for the -devel
sub-package. That is fine.

But, as lapack-devel and blas-devel include only libraries (and not header
files), I would exclude the correspondent dependency from the 'Requires:' part.
The same way, I would exclude dependency on 'libstdc++-devel'.

So, following is a suggestion for the corresponding part in the RPM
specification file:
-------------------
# The header files of %{name} include some Boost header files, delivered within
# the boost-devel sub-package. However, since there is no dependency on any
# Boost library (most of Boost is delivered as header files only), the RPM
# building process does not detect that dependency. That dependency must
# therefore be added manually.
Requires:       boost-devel
-------------------

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