https://bugzilla.redhat.com/show_bug.cgi?id=1183193 Rich Mattes <richmattes@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? --- Comment #13 from Rich Mattes <richmattes@xxxxxxxxx> --- It looks like you covered everything but the epel-specific constructs, which is fine if you're targeting centos as well as fedora (which it looks like you are) Unfortunately, the tests are failing: http://koji.fedoraproject.org/koji/taskinfo?taskID=9207143 It looks like it might have to do with the test that got re-enabled with the patch, and some CMake configuration issues. It appears that CMake isn't picking up the suitesparse headers: -- Failed to find CXSparse - Could not find CXSparse include directory, set CXSPARSE_INCLUDE_DIR to directory containing cs.h You need to set -DCXSPARSE_INCLUDE_DIR=%{_includedir}/suitesparse in the call to cmake. Once you do that, CMake reports that there's no BLAS implementation available, and that it needs one to build with suitesparse: -- A library with BLAS API not found. Please specify library location. The ATLAS library provides its BLAS routines in libsatlas.so, which CMake's FindBLAS apparently doesn't know to look for. You have to set -DBLAS_LIBRARIES to the ATLAS blas lib: %{_libdir}/atlas/libsatlas.so Once you do all that, CMake doesn't complain about missing deps, and the tests pass (in my local mockbuilds) (ref: https://groups.google.com/forum/#!topic/ceres-solver/-9fdnMBtuKI) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review