diff options
author | pukkandan <pukkandan.ytdlp@gmail.com> | 2021-09-24 05:51:54 +0530 |
---|---|---|
committer | pukkandan <pukkandan.ytdlp@gmail.com> | 2021-09-24 06:05:35 +0530 |
commit | b19404591a8ad4d0c7e962931ea809221e3f0b8e (patch) | |
tree | 56fb96572fb415d6d1858e7bbb6feea2ce088125 /yt_dlp/postprocessor/xattrpp.py | |
parent | 1f8471e22cdb5181aa19b0c63523ad96891ea2dd (diff) | |
download | hypervideo-pre-b19404591a8ad4d0c7e962931ea809221e3f0b8e.tar.lz hypervideo-pre-b19404591a8ad4d0c7e962931ea809221e3f0b8e.tar.xz hypervideo-pre-b19404591a8ad4d0c7e962931ea809221e3f0b8e.zip |
Separate the options `--ignore-errors` and `--no-abort-on-error`
In youtube-dl, `-i` ignores both download and post-processing error, and
treats the download as successful even if the post-processor fails.
yt-dlp used to skip the entire video on either error and there was no
option to ignore the post-processing errors like youtube-dl does.
By splitting the option into two, now either just the download errors
(--no-abort-on-error, default on CLI) or all errors (--ignore-errors)
can be ignored as per the users' needs
Closes #893
Diffstat (limited to 'yt_dlp/postprocessor/xattrpp.py')
-rw-r--r-- | yt_dlp/postprocessor/xattrpp.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/yt_dlp/postprocessor/xattrpp.py b/yt_dlp/postprocessor/xattrpp.py index 3d31f0ce5..95afa1c4f 100644 --- a/yt_dlp/postprocessor/xattrpp.py +++ b/yt_dlp/postprocessor/xattrpp.py @@ -57,8 +57,7 @@ class XAttrMetadataPP(PostProcessor): return [], info except XAttrUnavailableError as e: - self.report_error(str(e)) - return [], info + raise PostProcessingError(str(e)) except XAttrMetadataError as e: if e.reason == 'NO_SPACE': @@ -74,5 +73,5 @@ class XAttrMetadataPP(PostProcessor): msg += 'You need to use NTFS.' else: msg += '(You may have to enable them in your /etc/fstab)' - self.report_error(msg) + raise PostProcessingError(str(e)) return [], info |