majomi/intro-skipper
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Still check for ffmpeg errors on load

Browse Source
This commit is contained in:
TwistedUmbrellaX 2024-03-07 17:08:15 -05:00
parent b9cad22e3b
commit 4b353bc024
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ConfusedPolarBear.Plugin.IntroSkipper/Plugin.cs
View File

@ -109,6 +109,8 @@ public class Plugin : BasePlugin<PluginConfiguration>, IHasWebPages
_logger.LogError("Unknown error encountered while adding skip button: {Error}", ex); _logger.LogError("Unknown error encountered while adding skip button: {Error}", ex);
} }
} }
FFmpegWrapper.CheckFFmpegVersion();
} }
/// <summary> /// <summary>

3
ConfusedPolarBear.Plugin.IntroSkipper/ScheduledTasks/BaseItemAnalyzerTask.cs
View File

@ -53,8 +53,9 @@ public class BaseItemAnalyzerTask
IProgress<double> progress, IProgress<double> progress,
CancellationToken cancellationToken) CancellationToken cancellationToken)
{ {
var ffmpegError = FFmpegWrapper.CheckFFmpegVersion();
// Assert that ffmpeg with chromaprint is installed // Assert that ffmpeg with chromaprint is installed
if (Plugin.Instance!.Configuration.UseChromaprint && !FFmpegWrapper.CheckFFmpegVersion()) if (Plugin.Instance!.Configuration.UseChromaprint && !ffmpegError)
{ {
throw new FingerprintException( throw new FingerprintException(
"Analysis terminated! Chromaprint is not enabled in the current ffmpeg. If Jellyfin is running natively, install jellyfin-ffmpeg5. If Jellyfin is running in a container, upgrade to version 10.8.0 or newer."); "Analysis terminated! Chromaprint is not enabled in the current ffmpeg. If Jellyfin is running natively, install jellyfin-ffmpeg5. If Jellyfin is running in a container, upgrade to version 10.8.0 or newer.");