Bug 1010368

Summary: suitesparse-devel is missing c++ headers (with suffix 'hpp')
Product: Red Hat Enterprise Linux 7 Reporter: Nils Philippsen <nphilipp>
Component: suitesparseAssignee: Nils Philippsen <nphilipp>
Status: CLOSED CURRENTRELEASE QA Contact: Branislav Náter <bnater>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.0CC: bnater, brian, c.david86, dakingun, nphilipp, taylor, ttomecek, xjakub
Target Milestone: beta   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1001943 Environment:
Last Closed: 2014-06-13 09:29:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 1001869    
Bug Blocks: 1009360    

Description Nils Philippsen 2013-09-20 15:58:38 UTC
+++ This bug was initially created as a clone of Bug #1001943 +++
+++ This bug was initially created as a clone of Bug #1001869 +++

Description of problem:

The suitesparse-devel package does not include the C++ header files.

Version-Release number of selected component (if applicable):

Tested with suitesparse 3.4.0-7.el6 

How reproducible:

Always

Steps to Reproduce:
$ repoquery --list suitesparse-devel | grep hpp

Actual results:

Nothing is found.

Expected results:

I expect to find the hpp files from the source.

$ wget --quiet 'https://www.cise.ufl.edu/research/sparse/SuiteSparse/SuiteSparse-3.4.0.tar.gz'
$ tar tf SuiteSparse-3.4.0.tar.gz | grep hpp
SuiteSparse/SPQR/MATLAB/spqr_mx.hpp
SuiteSparse/SPQR/Include/SuiteSparseQR.hpp
SuiteSparse/SPQR/Include/spqr.hpp


Additional info:

Without SuiteSparseQR.hpp you cannot compile https://code.google.com/p/ceres-solver/

--- Additional comment from Tomas Tomecek on 2013-08-28 03:54:34 EDT ---

The bug is in spec file. This patch should fix it:

diff --git a/suitesparse.spec b/suitesparse.spec
index 770f727..b5b52c1 100644
--- a/suitesparse.spec
+++ b/suitesparse.spec
@@ -337,7 +337,7 @@ pushd Lib
   done
 popd
 pushd Include
-  for f in *.h;  do
+  for f in *.h*;  do
     cp -a $f ${RPM_BUILD_ROOT}%{_includedir}/%{name}/$f
   done
 popd

--- Additional comment from Nils Philippsen on 2013-09-03 05:34:20 EDT ---

I've kicked off builds for suitesparse-4.0.2-6 in Rawhide and F-20.

Comment 3 Branislav Náter 2014-04-08 15:57:49 UTC
Verified on package suitesparse-4.0.2-10.el7 using automated test. hpp header files are present.

::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: Test
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

:: [   PASS   ] :: Checking suitesparse-devel package list (Expected 0, got 0)
:: [   PASS   ] :: File 'output.log' should contain 'SuiteSparseQR\.hpp' 
:: [   PASS   ] :: File 'output.log' should contain 'spqr\.hpp' 
:: [   LOG    ] :: Duration: 1s
:: [   LOG    ] :: Assertions: 3 good, 0 bad
:: [   PASS   ] :: RESULT: Test

Comment 4 Ludek Smid 2014-06-13 09:29:49 UTC
This request was resolved in Red Hat Enterprise Linux 7.0.

Contact your manager or support representative in case you have further questions about the request.