Bug 1001869

Summary:	suitesparse-devel is missing c++ headers
Product:	Red Hat Enterprise Linux 6	Reporter:	Brian Pitts <brian>
Component:	suitesparse	Assignee:	Tomas Tomecek <ttomecek>
Status:	CLOSED ERRATA	QA Contact:	Jan Kepler <jkejda>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	6.4	CC:	jkejda, mkyral, ovasik, taylor
Target Milestone:	beta
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	suitesparse-3.4.0-9.el6	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1001943 (view as bug list)		Environment:
Last Closed:	2014-03-17 08:41:26 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:
Embargoed:
Bug Depends On:
Bug Blocks:	1001943, 1010368

Description Brian Pitts 2013-08-28 02:36:04 UTC

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/

Comment 1 Tomas Tomecek 2013-08-28 08:18:11 UTC

Thank you for bugreport, Brian.

The bug occurs in spec file: files with suffix 'hpp' are not being copied to %{_includedir} which is a mistake.

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

Comment 3 Tomas Tomecek 2014-03-10 16:11:16 UTC

$ rpm -ql -p suitesparse-devel-3.4.0-9.el6.x86_64.rpm | grep hpp
/usr/include/suitesparse/SuiteSparseQR.hpp
/usr/include/suitesparse/spqr.hpp

Comment 8 errata-xmlrpc 2014-03-17 08:41:26 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2014-0296.html