Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#60367 closed defect (fixed)

vtk 8.2.0 and VTK_INSTALL_PYTHON_MODULE_DIR

Reported by: bernard-giroux (Bernard Giroux) Owned by: stromnov (Andrey Stromnov)
Priority: Normal Milestone:
Component: ports Version: 2.6.2
Keywords: catalina Cc: majoc-at-astro (majoc-at-astro), jjstickel (Jonathan Stickel), ryandesign (Ryan Carsten Schmidt)
Port: vtk

Description (last modified by ryandesign (Ryan Carsten Schmidt))

Hi,

After upgrading vtk to 8.2.0 (with python 3.7), I can no longer import vtk. I found vtk.py in /opt/local/lib/python3.7/site-packages while all other modules are in /opt/local/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages

It look likes VTK_INSTALL_PYTHON_MODULE_DIR is no longer defined the portfile, e.g.

-DVTK_INSTALL_PYTHON_MODULE_DIR=${frameworks_dir}/Python.framework/Versions/3.7/lib/python3.7/site-packages

has been removed. Was this intentional?

Change History (11)

comment:1 Changed 5 years ago by jmroot (Joshua Root)

Cc: stromnov@… removed
Owner: set to stromnov
Status: newassigned

comment:2 Changed 5 years ago by majoc-at-astro (majoc-at-astro)

Cc: majoc-at-astro added

comment:3 Changed 5 years ago by majoc-at-astro (majoc-at-astro)

Hi there: we're getting a failure to build vtk at all with +python38 +qt5 +openmpi, in our regular build-from-scratch scripts, in multiple macOS versions. Should I submit that as a separate ticket? with cross-linking, naturally. Thanks in advance.

comment:4 in reply to:  3 Changed 5 years ago by majoc-at-astro (majoc-at-astro)

Replying to majoc-at-astro:

Hi there: we're getting a failure to build vtk at all with +python38 +qt5 +openmpi, in our regular build-from-scratch scripts, in multiple macOS versions. Should I submit that as a separate ticket? with cross-linking, naturally. Thanks in advance.

On second thoughts, our problem looks different enough to warrant its own ticket. Please see 60381.

comment:5 Changed 5 years ago by jjstickel (Jonathan Stickel)

Cc: jjstickel added

comment:6 Changed 5 years ago by rubendibattista (Ruben Di Battista)

Hello, I think I removed that by error in my last PR. I'll try to fix it...

comment:7 in reply to:  6 Changed 5 years ago by jjstickel (Jonathan Stickel)

Replying to rubendibattista:

Hello, I think I removed that by error in my last PR. I'll try to fix it...

I've been trying a couple things. VTK_INSTALL_PYTHON_MODULE_DIR seems to be ignored now. I found VTK_PYTHON_SITE_PACKAGES_SUFFIX but using:

-DVTK_PYTHON_SITE_PACKAGES_SUFFIX:PATH=${frameworks_dir}/Python.framework/Versions/${python_branch}/lib/python${python_branch}/site-packages

results in the files going int /opt/local/lib/opt/local/Library/... That is where I am at.

comment:8 Changed 5 years ago by jjstickel (Jonathan Stickel)

I just came across this:

https://public.kitware.com/pipermail/vtkusers/2019-January/103378.html

Looks like they added an S to MODULE, so it should be VTK_INSTALL_PYTHON_MODULES_DIR. That's kind of crappy. Testing now.

comment:9 Changed 5 years ago by jjstickel (Jonathan Stickel)

Resolution: fixed
Status: assignedclosed

In 271d37672d376a74408736692f13369616482955/macports-ports (master):

vtk: fix install location of python files (https://github.com/macports/macports-ports/pull/6926)

  • read configuration path variable for python variants
  • Closes: #60367

comment:10 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign added
Description: modified (diff)

Isn't this a runtime problem (rather than a build-time problem)? Doesn't the fix change the contents of files that get installed? Doesn't that mean the revision needs to be increased?

comment:11 in reply to:  10 Changed 5 years ago by jjstickel (Jonathan Stickel)

Replying to ryandesign:

Isn't this a runtime problem (rather than a build-time problem)? Doesn't the fix change the contents of files that get installed? Doesn't that mean the revision needs to be increased?

Good point. Sorry to miss it. We'll try to get the revision bumped in another PR.

Note: See TracTickets for help on using tickets.