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

Windows build fix against prebuilt package #5957

Merged
merged 2 commits into from
Jun 1, 2021
Merged

Windows build fix against prebuilt package #5957

merged 2 commits into from
Jun 1, 2021

Conversation

ilya-lavrenov
Copy link
Contributor

Details:

  • Looks like we need lib files for direct dependencies of onnx_importer, inference_engine as well.

Tickets:

@ilya-lavrenov ilya-lavrenov added the category: build OpenVINO cmake script / infra label Jun 1, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2021.4 milestone Jun 1, 2021
@ilya-lavrenov ilya-lavrenov changed the title Windows build fix Windows build fix against prebuilt package Jun 1, 2021
@ilya-lavrenov ilya-lavrenov marked this pull request as ready for review June 1, 2021 14:10
@ilya-lavrenov ilya-lavrenov requested a review from a team June 1, 2021 14:10
@azhogov azhogov merged commit 53ca9b2 into openvinotoolkit:master Jun 1, 2021
@ilya-lavrenov ilya-lavrenov deleted the windows-build-fix branch June 1, 2021 15:23
yekruglov pushed a commit to yekruglov/openvino that referenced this pull request Jun 7, 2021
@ilya-lavrenov @yekruglov
Windows build fix against prebuilt package (openvinotoolkit#5957)
7ca0f33 
* Revert "Fixed nGraph lib files for Windows (openvinotoolkit#5904)"

This reverts commit 442bb4e.

* Added inference_engine_transformations.lib back on Windows
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@ilya-lavrenov
Windows build fix against prebuilt package (openvinotoolkit#5957)
44ac182 
* Revert "Fixed nGraph lib files for Windows (openvinotoolkit#5904)"

This reverts commit 442bb4e.

* Added inference_engine_transformations.lib back on Windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@azhogov azhogov azhogov approved these changes

@GlebKazantaev GlebKazantaev Awaiting requested review from GlebKazantaev

@ilyachur ilyachur Awaiting requested review from ilyachur

Assignees
No one assigned
Labels
category: build OpenVINO cmake script / infra
Projects
None yet
Milestone
2021.4
Development

Successfully merging this pull request may close these issues.
2 participants
@ilya-lavrenov @azhogov