Skip to content

Update BUILDING-FEDORA.md#839

Open
samnooij wants to merge 1 commit into
royshil:mainfrom
samnooij:patch-1
Open

Update BUILDING-FEDORA.md#839
samnooij wants to merge 1 commit into
royshil:mainfrom
samnooij:patch-1

Conversation

@samnooij
Copy link
Copy Markdown

I needed to install onnxruntime-devel (and onnxruntime) as well before I got it to work on Fedora 43 (with GNOME).

Pull Request Checklist

Please read our latest CONTRIBUTING.md.

  • I have read the latest CONTRIBUTING.md.
  • I have acknowledged the licensing and patent grant policy.
  • I have included the required license headers.
  • I am confident with my code integrity.
  • I have signed off and verified all my commits.

I needed to install onnxruntime-devel (and onnxruntime) as well before I got it to work on Fedora 43 (with GNOME).
@umireon umireon added the auto-review Turn on this when PR is ready label Mar 24, 2026
@umireon umireon self-assigned this Mar 24, 2026
@umireon umireon requested a review from Copilot March 24, 2026 11:26
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

Updates the Fedora build documentation to reflect an additional dependency required to compile the OBS background removal plugin on newer Fedora releases.

Changes:

  • Add onnxruntime-devel to the documented dnf install dependency list on Fedora.

Comment thread docs/BUILDING-FEDORA.md
Comment on lines 20 to +22
obs-studio-devel \
opencv-devel
opencv-devel \
onnxruntime-devel
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

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

The PR description says Fedora needed both onnxruntime-devel and onnxruntime, but the updated dependency list only adds onnxruntime-devel. To avoid confusion (and to ensure the runtime library is present if -devel doesn’t pull it in on some Fedora variants), either add onnxruntime here as well or explicitly note that installing onnxruntime-devel will install the runtime package as a dependency.

Copilot uses AI. Check for mistakes.
@umireon umireon added auto-review Turn on this when PR is ready and removed auto-review Turn on this when PR is ready labels Mar 24, 2026
@umireon umireon added this to the 1.4.1 milestone Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-review Turn on this when PR is ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants