View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014700VTK(No Category)public2014-04-17 17:392015-08-11 19:04
ReporterOrion Poplawski 
Assigned ToT.J. Corona 
PrioritylowSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0014700: Need to remove old vtkExodusIIReaderVariableCheck.h?
DescriptionNothing seems to reference vtkExodusIIReaderVariableCheck.h and it tries to include the non-installed (but existing) vtkExodusIIReaderPrivate.h.
Tagshackaton
ProjectTBD
Typeincorrect functionality
Attached Filespatch file icon headers.patch [^] (1,789 bytes) 2014-07-10 13:19 [Show Content]

 Relationships

  Notes
(0033004)
Orion Poplawski (reporter)
2014-07-10 13:16

There seem to be a number of header files that are not installed but referenced by installed headers.

See https://bugzilla.redhat.com/show_bug.cgi?id=1100911 [^]

$ 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

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).
(0034964)
T.J. Corona (developer)
2015-08-11 19:04

vtkExodusIIReaderVariableCheck.h is now excluded from the install. vtk3DS.h, vtkMPIImageReader.h
and vtkProjectedPolyDataRayBounder.h are now included in VTK's install. vtkSIVectorPropertyTemplate.h and vtkMaterialInterfaceIdListItem.h are now included in Paraview's install. For more information, see:

https://gitlab.kitware.com/vtk/vtk/commit/a50992fa614b75a425406b45a9aed47f401a8282 [^]
https://gitlab.kitware.com/vtk/vtk/commit/4edebb396efc533b6c2124a303f54c1df271b703 [^]
https://gitlab.kitware.com/paraview/paraview/commit/38112d6d04d86b7404b8170b0d5ec458911f8912 [^]

 Issue History
Date Modified Username Field Change
2014-04-17 17:39 Orion Poplawski New Issue
2014-07-10 13:16 Orion Poplawski Note Added: 0033004
2014-07-10 13:19 Orion Poplawski File Added: headers.patch
2014-10-01 12:45 Berk Geveci Tag Attached: hackaton
2014-10-01 12:45 Berk Geveci Assigned To => Berk Geveci
2014-10-01 12:45 Berk Geveci Status backlog => tabled
2014-10-02 15:32 Andy Bauer Assigned To Berk Geveci => Andy Bauer
2014-10-02 15:32 Andy Bauer Status tabled => backlog
2014-10-02 16:40 Andy Bauer Assigned To Andy Bauer =>
2014-10-02 16:40 Andy Bauer Assigned To => Berk Geveci
2015-08-05 10:26 Utkarsh Ayachit Assigned To Berk Geveci => T.J. Corona
2015-08-11 19:04 T.J. Corona Note Added: 0034964
2015-08-11 19:04 T.J. Corona Status backlog => closed
2015-08-11 19:04 T.J. Corona Resolution open => fixed


Copyright © 2000 - 2018 MantisBT Team