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

openvino: added OpenVINO 1.0 API headers for older releases #22289

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

ilya-lavrenov
Copy link
Contributor

Specify library name and version: lib/1.0

Required for OpenCV #21204 since it still uses old API in some places


@ghost
Copy link

ghost commented Jan 13, 2024

I detected other pull requests that are modifying openvino/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot

This comment has been minimized.

@ilya-lavrenov
Copy link
Contributor Author

Blocked by #22281

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@ilya-lavrenov
Copy link
Contributor Author

@RubenRBS could you please help? What is wrong with Conan v2 CI? I cannot access the link and the status is red.

@ilya-lavrenov
Copy link
Contributor Author

CC @SpaceIm

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 10 (5d3cf2c5379b1bf4f7ae6e4ff8dba37805006bdb):

  • openvino/2023.2.0:
    All packages built successfully! (All logs)

  • openvino/2023.1.0:
    All packages built successfully! (All logs)


Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 10 (5d3cf2c5379b1bf4f7ae6e4ff8dba37805006bdb):

  • openvino/2023.1.0:
    All packages built successfully! (All logs)

  • openvino/2023.2.0:
    All packages built successfully! (All logs)

@ilya-lavrenov
Copy link
Contributor Author

@uilianries @RubenRBS
Could you please approve and merge?

Copy link
Member

@uilianries uilianries left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@uilianries uilianries self-assigned this Jan 26, 2024
@ilya-lavrenov
Copy link
Contributor Author

license/cla

@uilianries do I need to care about this handing check?
I've contributed to conan-center-index numerous times and I suppose CLA checks should be Ok with me :)

@uilianries
Copy link
Member

license/cla

@uilianries do I need to care about this handing check? I've contributed to conan-center-index numerous times and I suppose CLA checks should be Ok with me :)

@ilya-lavrenov sometimes the CLA bot gets stuck, so just need to push it again. Fortunately, anyone can do it. In this case just need a HTTP GET to https://cla-assistant.io/check/conan-io/conan-center-index?pullRequest=22289

@conan-center-bot conan-center-bot merged commit 65499b3 into conan-io:master Jan 26, 2024
18 checks passed
@ilya-lavrenov ilya-lavrenov deleted the openvino-old-api branch January 26, 2024 20:04
Ahajha pushed a commit to Ahajha/conan-center-index that referenced this pull request Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants