-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
openni2: new recipe #23790
base: master
Are you sure you want to change the base?
openni2: new recipe #23790
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 2a7f72aopenni2/2.2.0.33@#5456def82afbdb5f0c3f109c9e012bbe
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit df26ce5openni2/2.2.0.33@#1cc08ebe904ca7d397c4dc25ada30fe2
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 6dd9d3eopenni2/2.2.0.33@#fecc3dcc0deb22ae24e9061841a8fcda
|
Conan v1 pipeline ✔️All green in build 8 (
Conan v2 pipeline ✔️
All green in build 8 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! Thanks a lot for the contribution.
This looks like a fork of https://github.com/OpenNI/OpenNI2 so I think it should be named structureio-openni2
That's a fair point, but the old repo is pretty much dead and the fork is used as the main repo by all other package managers, including Debian/Ubuntu: https://repology.org/project/openni2/information |
Hi @valgur, thanks for the info, I was digging a bit and looks like the own owners of the original repo recommend using the fork, so I think it's ok for the recipe to be named openni2 but please let's keep a comment in the recipe stating that this is the recommended fork from the original repo. |
Adds OpenNI2: https://github.com/structureio/OpenNI2
The OpenNI 2.0 API provides access to PrimeSense-compatible depth sensors.
An optional dependency for the following packages on CCI: