Skip to content

Do not export CMAKE_..._OUTPUT_DIRECTORY variables from developer package cmake #32319

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
Merged
merged 2 commits into from
Oct 13, 2025
Merged

Do not export CMAKE_..._OUTPUT_DIRECTORY variables from developer package cmake #32319

merged 2 commits into from
Oct 13, 2025

Conversation

olpipi
Copy link
Contributor

@olpipi olpipi commented Oct 7, 2025 •
edited
Loading

Details:

  • Unset CMAKE_..._OUTPUT_DIRECTORY for developer package to not export these values for find_package(OpenVINODeveloperPackage)
  • Add WA for wheels build as it requires these variables

Tickets:

@olpipi olpipi requested review from a team as code owners October 7, 2025 14:08
@olpipi olpipi requested review from a team as code owners October 7, 2025 14:08
@olpipi olpipi requested review from mlukasze and removed request for a team October 7, 2025 14:08
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: GPU OpenVINO GPU plugin category: build OpenVINO cmake script / infra category: Python API OpenVINO Python bindings category: samples OpenVINO Runtime Samples category: dependency_changes Pull requests that update a dependency file category: packaging OpenVINO packaging / distribution category: NPU OpenVINO NPU plugin labels Oct 7, 2025
@olpipi olpipi force-pushed the cmake_output_dirs_fix branch from b55ee01 to 06558cf Compare October 7, 2025 14:44
@github-actions github-actions bot added category: docs OpenVINO documentation and removed category: inference OpenVINO Runtime library - Inference category: GPU OpenVINO GPU plugin category: Python API OpenVINO Python bindings category: packaging OpenVINO packaging / distribution category: NPU OpenVINO NPU plugin labels Oct 7, 2025
@olpipi olpipi force-pushed the cmake_output_dirs_fix branch from 06558cf to 532d7f7 Compare October 7, 2025 14:56
@github-actions github-actions bot added the category: packaging OpenVINO packaging / distribution label Oct 7, 2025
@olpipi olpipi changed the title [DO NOT REVIEW] Remove global CMAKE_..._OUTPUT_DIRECTORY Remove global CMAKE_..._OUTPUT_DIRECTORY Oct 9, 2025
@olpipi olpipi force-pushed the cmake_output_dirs_fix branch from 532d7f7 to cb3c9e7 Compare October 10, 2025 12:04
@github-actions github-actions bot removed category: samples OpenVINO Runtime Samples category: docs OpenVINO documentation category: dependency_changes Pull requests that update a dependency file category: packaging OpenVINO packaging / distribution labels Oct 10, 2025
@github-actions github-actions bot added the category: Python API OpenVINO Python bindings label Oct 10, 2025
@olpipi olpipi changed the title Remove global CMAKE_..._OUTPUT_DIRECTORY Do not export CMAKE_..._OUTPUT_DIRECTORY variables from developer package cmake Oct 10, 2025
@praasz praasz added this to the 2025.4 milestone Oct 13, 2025
@olpipi olpipi added this pull request to the merge queue Oct 13, 2025
Merged via the queue into openvinotoolkit:master with commit f338451 Oct 13, 2025
208 checks passed
@olpipi olpipi deleted the cmake_output_dirs_fix branch October 13, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mryzhov mryzhov mryzhov approved these changes

@praasz praasz praasz approved these changes

@mlukasze mlukasze Awaiting requested review from mlukasze mlukasze was automatically assigned from openvinotoolkit/openvino-maintainers

Assignees

No one assigned

Labels

category: build OpenVINO cmake script / infra category: Python API OpenVINO Python bindings

Projects

None yet

Milestone

2025.4

Development

Successfully merging this pull request may close these issues.

3 participants

@olpipi @mryzhov @praasz