Skip to content

fix(opencv): Disable openni support#16612

Merged
christopherco merged 1 commit intotomls/base/mainfrom
chrco/fix-opencv2
Apr 12, 2026
Merged

fix(opencv): Disable openni support#16612
christopherco merged 1 commit intotomls/base/mainfrom
chrco/fix-opencv2

Conversation

@christopherco
Copy link
Copy Markdown
Collaborator

openni-devel and openni-primesense are not available in Azure Linux. OpenNI is an old (v1.5.7) Kinect/PrimeSense depth sensor library not relevant for a cloud/server distro. Use the spec's existing bcond to disable it via build.without.

openni-devel and openni-primesense are not available in Azure Linux.
OpenNI is an old (v1.5.7) Kinect/PrimeSense depth sensor library not
relevant for a cloud/server distro. Use the spec's existing bcond to
disable it via build.without.
@christopherco christopherco changed the title fix: Disable openni support in opencv fix(opencv): Disable openni support Apr 12, 2026
@christopherco christopherco marked this pull request as ready for review April 12, 2026 06:17
Copilot AI review requested due to automatic review settings April 12, 2026 06:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Disables OpenNI support for the opencv component in Azure Linux by leveraging the upstream spec’s existing openni bcond, avoiding dependencies that are unavailable in the distro.

Changes:

  • Add an opencv build configuration to pass --without openni (via build.without).
  • Document rationale for disabling OpenNI in the component definition.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@christopherco christopherco merged commit 3484cfd into tomls/base/main Apr 12, 2026
9 checks passed
@christopherco christopherco deleted the chrco/fix-opencv2 branch April 12, 2026 06:39
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.

3 participants