linuxserver / docker-radarr

GNU General Public License v3.0
650 stars 104 forks source link

Use latest ffprobe #220

Closed zestysoft closed 7 months ago

zestysoft commented 7 months ago

I recently ran into an issue that's being caused by the outdated ffprobe binary that's included in the containers here.

This PR uses a more recent version of the binary which solved my problem. Sharing here in case it's a welcome change.

I also bumped the alpine base and added some missing binaries for the apk add command.

This goes into more detail about the problem I ran into: https://www.reddit.com/r/radarr/comments/1aog9bh/outdated_ffprobe_binary_failing_to_probe_media/

thespad commented 7 months ago

Overriding the version of ffprobe that is bundled with Radarr would put us in an unsupported configuration; if you're having issues with the shipped version I'd suggest opening an issue with the Radarr project - or you can customise the container at runtime.

In addition, curl and jq are included in our base images, which is why we don't install them again downstream.

LinuxServer-CI commented 7 months ago
I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/radarr/5.2.6.8376-pkg-9722da3a-dev-90a919ec159e0c8e264e9093def3cc1101cabc6f-pr-220/index.html https://ci-tests.linuxserver.io/lspipepr/radarr/5.2.6.8376-pkg-9722da3a-dev-90a919ec159e0c8e264e9093def3cc1101cabc6f-pr-220/shellcheck-result.xml Tag Passed
amd64-5.2.6.8376-pkg-9722da3a-dev-90a919ec159e0c8e264e9093def3cc1101cabc6f-pr-220
arm64v8-5.2.6.8376-pkg-9722da3a-dev-90a919ec159e0c8e264e9093def3cc1101cabc6f-pr-220