Skip to content

Conversation

stevelr
Copy link
Contributor

@stevelr stevelr commented Aug 23, 2023

On macos, also look in homebrew installation folders: $HOMEBREW_PREFIX/lib and /opt/homebrew/lib

Also fixed a couple clippy errors and a build error in xtask.

Copy link
Contributor

@abrown abrown left a comment

Choose a reason for hiding this comment

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

@stevelr, good to see you! I think this change makes sense in general and I believe you can safely ignore the dependency issues (I'll have to update some crates somewhere). But I did want to make one onerous request: is there any way you can add a CI task to install OpenVINO via homebrew in a GitHub runner so we can check this out? If it's too difficult for some reason feel free to let me know, but if you are able to get something like that working we can use it to check the functionality here and eventually even move it into https://github.com/abrown/install-openvino-action.

@stevelr
Copy link
Contributor Author

stevelr commented Oct 6, 2023

As of e334a47, this issue is fixed.

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.

2 participants