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.

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

Fix itt include dir detection #748

Conversation

emmanuelattia
Copy link
Contributor

ITT_INCLUDE_DIR was not properly detected when VTune is on the machine.

@emmanuelattia emmanuelattia requested a review from a team June 3, 2020 13:25
@ilya-lavrenov ilya-lavrenov self-assigned this Jun 3, 2020
@ilya-lavrenov ilya-lavrenov added the category: build OpenVINO cmake script / infra label Jun 3, 2020
@ilya-lavrenov ilya-lavrenov added this to the 2020.4 milestone Jun 3, 2020
@emmanuelattia
Copy link
Contributor Author

I have no idea why ci/jenkins failed, i don't have access to the details

@asuhov
Copy link
Contributor

asuhov commented Jun 3, 2020

Functional tests for CPU plugin failed. It seems it's a sporadic issue, because they passed after the second run.

@ilya-lavrenov
Copy link
Contributor

Reverted the patch which triggers sporadic behavior #756

@ilya-lavrenov
Copy link
Contributor

@emmanuelattia-philips sorry for inconvenience. Could you please rebase and the tests will be run once again?

Rebase upstream -> master -> fix_tt_include_dir_detection
@emmanuelattia
Copy link
Contributor Author

Please notify me when i should retry the azure devops windows seems broken (or big mistake from me ?)

@ilya-lavrenov
Copy link
Contributor

ilya-lavrenov commented Jun 3, 2020

seems another issues with azure service provider.. (see microsoft/azure-pipelines-tasks#13044) Sure, will notify.

@ilya-lavrenov ilya-lavrenov merged commit a0d1dae into openvinotoolkit:master Jun 5, 2020
@AnastasiaKazantaeva AnastasiaKazantaeva added the ExternalPR External contributor label Jun 9, 2020
@emmanuelattia emmanuelattia deleted the fix_itt_include_dir_detection branch November 18, 2020 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra ExternalPR External contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants