Bug 1100911

Summary: Paraview vtk headers include non-installed headers
Product: [Fedora] Fedora Reporter: Sandro Mani <manisandro>
Component: paraviewAssignee: Orion Poplawski <orion>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dakingun, orion, pertusus
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: paraview-4.1.0-7.fc22 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-22 15:07:10 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:
Attachments:
Description Flags
Patch
none
Patch none

Description Sandro Mani 2014-05-23 20:29:56 UTC
Description of problem:

$ cd /usr/include/paraview
$ for file in $(egrep -r '"vtk\w+\.h"' * | awk -F'"' '{print $2}' | sort | uniq); do if [ ! -e $file ]; then echo $file; fi; done

vtk3DS.h
vtkExodusIIReaderPrivate.h
vtkMaterialInterfaceIdListItem.h
vtkMPIImageReader.h
vtkProjectedPolyDataRayBounder.h
vtkSIVectorPropertyTemplate.h

Version-Release number of selected component (if applicable):
paraview-4.1.0-2.fc21.x86_64

Comment 1 Sandro Mani 2014-07-07 21:56:51 UTC
Created attachment 916204 [details]
Patch

I've found out how such headers are supposted to be installed, a patch is attached.

Some notes:

I suspect vtkExodusIIReaderVariableCheck.h (who references vtkExodusIIReaderPrivate.h) shouldn't actually be installed, so the part of the patch for vtkExodusIIReaderPrivate.h can probably be discarded. (The same situation exists in the vtk package.)

vtkMPIImageReader.h is part of paraview-openmpi-devel, so this header is not actually missing.

vtk3DS.h is also missing from the vtk package.

vtkProjectedPolyDataRayBounder.h simply does not exist (and same situation again in the vtk package).

Comment 2 Sandro Mani 2014-07-07 21:58:05 UTC
Created attachment 916206 [details]
Patch

Comment 3 Orion Poplawski 2014-07-10 17:20:17 UTC
Upstream vtkExodus bug: http://www.vtk.org/Bug/view.php?id=14700

I added your comments/patch there too, thanks.