Merge pull request #143 from maya-doshi/autodetect-ffmpeg
Autodetect ffmpeg path if not present in configuration
This commit is contained in:
@@ -75,12 +75,14 @@ class Download:
|
|||||||
if not self.settings.data.path_binary_ffmpeg and (
|
if not self.settings.data.path_binary_ffmpeg and (
|
||||||
self.settings.data.video_convert_mp4 or self.settings.data.extract_flac
|
self.settings.data.video_convert_mp4 or self.settings.data.extract_flac
|
||||||
):
|
):
|
||||||
|
self.settings.data.path_binary_ffmpeg = shutil.which("ffmpeg")
|
||||||
|
if not self.settings.data.path_binary_ffmpeg:
|
||||||
self.settings.data.video_convert_mp4 = False
|
self.settings.data.video_convert_mp4 = False
|
||||||
self.settings.data.extract_flac = False
|
self.settings.data.extract_flac = False
|
||||||
|
|
||||||
self.fn_logger.error(
|
self.fn_logger.error(
|
||||||
"FFmpeg is not set. Videos can be downloaded but will not be processed. FLAC cannot be extracted from MP4 containers. "
|
"FFmpeg is not set. Videos can be downloaded but will not be processed. FLAC cannot be extracted from MP4 containers. "
|
||||||
"Make sure FFmpeg is installed and the path to the binary is configured (`path_binary_ffmpeg`)."
|
"Make sure FFmpeg is installed. The path to the binary can be specified (`path_binary_ffmpeg`)."
|
||||||
)
|
)
|
||||||
|
|
||||||
def _download(
|
def _download(
|
||||||
|
|||||||
@@ -70,9 +70,9 @@ class HelpSettings:
|
|||||||
format_video: str = "Where to download videos and how to name the items."
|
format_video: str = "Where to download videos and how to name the items."
|
||||||
video_convert_mp4: str = (
|
video_convert_mp4: str = (
|
||||||
"Videos are downloaded as MPEG Transport Stream (TS) files. With this option each video "
|
"Videos are downloaded as MPEG Transport Stream (TS) files. With this option each video "
|
||||||
"will be converted to MP4. FFmpeg must be installed and the binary path must be configured."
|
"will be converted to MP4. FFmpeg must be installed."
|
||||||
)
|
)
|
||||||
path_binary_ffmpeg: str = "If you like to convert videos, this must point to your installed ffmpeg binary."
|
path_binary_ffmpeg: str = "Path to FFmpeg binary. Only necesarry if FFmpeg not present in path."
|
||||||
metadata_cover_dimension: str = (
|
metadata_cover_dimension: str = (
|
||||||
"The dimensions of the cover image embedded into the track. Possible values: 320x320, 640x640x 1280x1280."
|
"The dimensions of the cover image embedded into the track. Possible values: 320x320, 640x640x 1280x1280."
|
||||||
)
|
)
|
||||||
|
|||||||
Reference in New Issue
Block a user