[PR #4963] [CLOSED] Fix for 4933: FFMPeg detection. #10344

Closed
opened 2026-02-07 06:18:36 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4963
Author: @BaronGreenback
Created: 1/6/2021
Status: Closed

Base: masterHead: ffmpegFix


📝 Commits (10+)

  • 1921a36 fix
  • 5c656c9 Changed to bool
  • 98a4de7 changed comment
  • 68c4294 Update MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs
  • 99fe722 Update MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs
  • 41b8d6f Update MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs
  • 4092a4b Update tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs
  • b20a516 renamed method and variable names.
  • 157cb84 added warning
  • 0caeee1 Corrected tests

📊 Changes

3 files changed (+110 additions, -34 deletions)

View changed files

📝 MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs (+62 -14)
📝 MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs (+32 -19)
📝 tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs (+16 -1)

📄 Description

If a version of ffmpeg is discovered that is outside the recommended version - this continues to look for a better one.

Also fixes https://github.com/jellyfin/jellyfin/issues/4933, whereby a version being outside of the recommended results in no version being selected, as function returns false.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/4963 **Author:** [@BaronGreenback](https://github.com/BaronGreenback) **Created:** 1/6/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `ffmpegFix` --- ### 📝 Commits (10+) - [`1921a36`](https://github.com/jellyfin/jellyfin/commit/1921a36af6a6f2fd9ef3e5676785d98ecb89a247) fix - [`5c656c9`](https://github.com/jellyfin/jellyfin/commit/5c656c9cc8ae21f410284506d6decb9db924a6a7) Changed to bool - [`98a4de7`](https://github.com/jellyfin/jellyfin/commit/98a4de72e5bcf5d33089a2a159ffd59937e72def) changed comment - [`68c4294`](https://github.com/jellyfin/jellyfin/commit/68c42949a90606b92afcfd09deb8a352f9b32d58) Update MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs - [`99fe722`](https://github.com/jellyfin/jellyfin/commit/99fe7221e070e49ba144449b9a4a085362662bbc) Update MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs - [`41b8d6f`](https://github.com/jellyfin/jellyfin/commit/41b8d6f6ac2ef15a2d6df62510d0f415198f6222) Update MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs - [`4092a4b`](https://github.com/jellyfin/jellyfin/commit/4092a4bf19ffb8431f3449a0eb1bc0a3be702d3e) Update tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs - [`b20a516`](https://github.com/jellyfin/jellyfin/commit/b20a5161cfb637620b4d33a757e475abc136db6a) renamed method and variable names. - [`157cb84`](https://github.com/jellyfin/jellyfin/commit/157cb84cbe5aaf96dd4195b0a2abf135cad6c4ef) added warning - [`0caeee1`](https://github.com/jellyfin/jellyfin/commit/0caeee18d7bf5549ff222899f8dcb14045afe3a3) Corrected tests ### 📊 Changes **3 files changed** (+110 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs` (+62 -14) 📝 `MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs` (+32 -19) 📝 `tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs` (+16 -1) </details> ### 📄 Description If a version of ffmpeg is discovered that is outside the recommended version - this continues to look for a better one. Also fixes https://github.com/jellyfin/jellyfin/issues/4933, whereby a version being outside of the recommended results in no version being selected, as function returns false. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:18:36 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10344