New comment by yopito on void-packages repository https://github.com/void-linux/void-packages/pull/29466#issuecomment-800659268 Comment: Comparing this PR with my freecad.current branch, I've noticed the points below. This is a partial review, I have to compare other items (some patches and template file too). `srcpkgs/freecad/patches/001-remove-python2.patch` is useless: don't need to build with python3. `srcpkgs/freecad/patches/150-vtk9.patch` : content is out of date. suggestion: ``` Taken from https://github.com/wwmayer/FreeCAD/commit/bb9bcbd51df7c3cb76c5823038e4ea0f7e25a9ff but removed patch on file src/Mod/Fem/Gui/ViewProviderFemPostObject.cpp : already applied (2021-01-20) ``` missing patch `srcpkgs/freecad/patches/152-cmake-vtk9.patch`: ``` --- cMake/FreeCAD_Helpers/SetupSalomeSMESH.cmake 2020-11-26 15:59:27.000000000 +0100 +++ cMake/FreeCAD_Helpers/SetupSalomeSMESH.cmake 2020-11-26 23:09:12.778262636 +0100 @@ -11,6 +11,8 @@ set(SMESH_VERSION_TWEAK 0) #if we use smesh we definitely also need vtk, no matter of external or internal smesh + find_package(VTK REQUIRED NO_MODULE) + if(${VTK_MAJOR_VERSION} LESS 9) set (VTK_COMPONENTS vtkCommonCore vtkCommonDataModel @@ -23,18 +25,37 @@ vtkFiltersSources vtkFiltersGeometry ) - - # check which modules are available - if(UNIX OR WIN32) find_package(VTK COMPONENTS vtkCommonCore REQUIRED NO_MODULE) list(APPEND VTK_COMPONENTS vtkIOMPIParallel vtkParallelMPI vtkhdf5 vtkFiltersParallelDIY2 vtkRenderingCore vtkInteractionStyle vtkRenderingFreeType vtkRenderingOpenGL2) + else() + # VTK 9 changed its component names + set (VTK_COMPONENTS + CommonCore + CommonDataModel + FiltersVerdict + IOXML + FiltersCore + FiltersGeneral + IOLegacy + FiltersExtraction + FiltersSources + FiltersGeometry + ) + find_package(VTK COMPONENTS CommonCore REQUIRED NO_MODULE) + list(APPEND VTK_COMPONENTS IOMPIParallel ParallelMPI hdf5 FiltersParallelDIY2 RenderingCore InteractionStyle RenderingFreeType RenderingOpenGL2) + endif() + + # check which modules are available foreach(_module ${VTK_COMPONENTS}) + if(${VTK_MAJOR_VERSION} LESS 9) list (FIND VTK_MODULES_ENABLED ${_module} _index) + else() + list (FIND VTK_AVAILABLE_COMPONENTS ${_module} _index) + endif() if (${_index} GREATER -1) list(APPEND AVAILABLE_VTK_COMPONENTS ${_module}) endif() endforeach() - endif() # don't check VERSION 6 as this would exclude VERSION 7 if(AVAILABLE_VTK_COMPONENTS) ```