Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixed pyi file build when OpenVINO_DIR is externally defined #1271

Merged
merged 2 commits into from
Nov 28, 2024
Merged

Fixed pyi file build when OpenVINO_DIR is externally defined #1271

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
83f071c
Fixed pyi file build when OpenVINO_DIR is externally defined
ilya-lavrenov Nov 28, 2024
e52dea7
Revert chnages in python source files
ilya-lavrenov Nov 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions src/python/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,21 @@ elseif(DEFINED PY_BUILD_CMAKE_PACKAGE_NAME AND NOT WIN32)
# in case of wheel build, pybind11-stubgen is always available via pyproject.toml's build-system
# except Win32 where we have issues with pybind11_stubgen executable which cannot import its own module
set(pybind11_stubgen_AVAILABLE ON)

# by default, wheel build is performed with build-isolation, which means that some variables like PYTHONPATH
# are not available. But if user called setupvars.sh, then OpenVINO dir is available, while PYTHONPATH - no.
# In this case, we will have mismatch on Linux when OpenVINO can point on build dir / install dir, while
# PYTHONPATH points out to locally installed tmp OpenVINO wheel (build against wheel).
# Ways to handle it:
# - setting PYTHONPATH to $ENV{INTEL_OPENVINO_DIR}/python if INTEL_OPENVINO_DIR is defined. It means we are building against
# OpenVINO archive or installation tree
# - if it's not defined, we cannot do any guesses and hence, disable pybind11-stubgen usage
if(DEFINED ENV{INTEL_OPENVINO_DIR})
set(openvino_pythonpath "$ENV{INTEL_OPENVINO_DIR}/python")
elseif(LINUX AND NOT OpenVINO_DIR STREQUAL OpenVINO_DIR_PY)
# here we imply that OpenVINO_DIR_PY points to manylinux, while OpenVINO_DIR point to Ubuntu binaries
set(pybind11_stubgen_AVAILABLE OFF)
endif()
endif()

# but we also need to check whether OpenVINO is installed
Expand Down
Loading