Skip to content

Commit

Permalink
media-gfx/openvdb: Use full path for Python_EXECUTABLE
Browse files Browse the repository at this point in the history
Openvdb fails to configure with numpy enabled when Python_EXECUTABLE is
set using ${EPYTHON} as it is unable to find the required python
components.

The cmake docs state that Python_EXECUTABLE must be set to the PATH
of the python interpreter.

See https://cmake.org/cmake/help/v3.16/module/FindPython.html

This means that it must be set to ${PYTHON} which contains the absolute
path in python-r1, rather than ${EPYTHON} which only contains the
executable name. Doing so resolves bug 738928.

Signed-off-by: Adrian Grigo <[email protected]>
Closes: https://bugs.gentoo.org/738928
Package-Manager: Portage-2.3.103, Repoman-2.3.23
Closes: gentoo#17310
Signed-off-by: Sam James <[email protected]>
  • Loading branch information
redchillipadi authored and thesamesam committed Sep 9, 2020
1 parent 6a2aacf commit fbc9f9a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion media-gfx/openvdb/openvdb-7.0.0-r1.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ src_configure() {
-DOPENVDB_BUILD_PYTHON_MODULE=ON
-DUSE_NUMPY=$(usex numpy)
-DPYOPENVDB_INSTALL_DIRECTORY="$(python_get_sitedir)"
-DPython_EXECUTABLE="${EPYTHON}"
-DPython_EXECUTABLE="${PYTHON}"
)
fi

Expand Down
2 changes: 1 addition & 1 deletion media-gfx/openvdb/openvdb-7.1.0-r1.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ src_configure() {
-DOPENVDB_BUILD_PYTHON_MODULE=ON
-DUSE_NUMPY=$(usex numpy)
-DPYOPENVDB_INSTALL_DIRECTORY="$(python_get_sitedir)"
-DPython_EXECUTABLE="${EPYTHON}"
-DPython_EXECUTABLE="${PYTHON}"
)
fi

Expand Down

0 comments on commit fbc9f9a

Please sign in to comment.