Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

ParaView example stopped building #1810

Merged
merged 4 commits into from
Feb 14, 2024
Merged

ParaView example stopped building #1810

merged 4 commits into from
Feb 14, 2024

Conversation

reidpr
Copy link
Collaborator

@reidpr reidpr commented Jan 17, 2024

The ParaView example stopped building sometime recently. The error is:

# -- Could NOT find Python3 (missing: Python3_INCLUDE_DIRS Development.Module) (found suitable version "3.11.5", minimum required is "3.2")
# CMake Error at VTK/CMake/vtkModule.cmake:4228 (message):
#   Could not find the Python3 external dependency.
# Call Stack (most recent call first):
#   VTK/Utilities/Python/CMakeLists.txt:49 (vtk_module_find_package)

I was able to track this down to the almalinux:8 image not being stable because it pulls in the AppStream repository. At some point this led to Python 3.11 being installed in addition to 3.6 from the base OS.

The changes in this branch seem to have successfully pointed ParaView to Python 3.11, but that yields a different error:

[  9%] Building CXX object Remoting/ClientServerStream/CMakeFiles/RemotingClientServerStream.dir/vtkClientServerInterpreterInitializer.cxx.o
In file included from /usr/local/src/ParaView-v5.9.1/VTK/ThirdParty/mpi4py/vtkmpi4py/src/MPI.c:4:
/usr/local/src/ParaView-v5.9.1/VTK/ThirdParty/mpi4py/vtkmpi4py/src/mpi4py.MPI.c:196:12: fatal error: longintrepr.h: No such file or directory
   #include "longintrepr.h"
            ^~~~~~~~~~~~~~~
compilation terminated.
[... lots more output ...]
make: *** [Makefile:136: all] Error 2
something went wrong, rolling back ...
HEAD is now at eb27f9de3 ARG PARAVIEW_VERSION='5.9.1'
error: build failed: RUN command exited with 2

Apparently this is because longintrepr.h is not actually a public header, and Python 3.10 started enforcing this (cython/cython#4461)

Disabling AppStream caused other failures I wasn’t able to track down.

I am giving up for now. This PR contains what I have so far.

I think there are two things to try next:

  1. Disable AppStream and any other bleeding-edge repos.
  2. Upgrade to the newest ParaView.

@reidpr reidpr added this to the 0.37 milestone Feb 14, 2024
@reidpr reidpr merged commit 17ae19b into master Feb 14, 2024
6 checks passed
@reidpr reidpr deleted the paraview-tests branch February 14, 2024 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants