Commit Graph

184 Commits (8d55e439491b937e08557a84036fce8d0c240cfb)

Author SHA1 Message Date
dirkf 21792b88b7 [external/FFmpeg] Fix and improve --ffmpeg-location handling
* pass YoutubeDL (FileDownloader) to FFmpegPostProcessor constructor
* consolidate path search in FFmpegPostProcessor
* make availability of FFmpegFD depend on existence of FFmpegPostProcessor
* detect ffmpeg executable on instantiation of FFmpegFD
* resolves #32735
7 months ago
dirkf a25e9f3c84 [compat] Use `compat_open()` 1 year ago
dirkf e52e8b8111 [postprocessor] Don't replace existing value with null metadata parsed from title 2 years ago
pukkandan 0700fde640 [utils, etc] Kill child processes when yt-dl is killed
* derived from PR #26592, closes #26592

Authored by: Unrud
2 years ago
kikuyan 3a7ef27cf3
[postprocessor/ffmpeg] Show ffmpeg output on error (refs #22680) (#29336) 3 years ago
Max 646052e416
[postprocessor/embedthumbnail] Recognize atomicparsley binary in lowercase (#28112) 4 years ago
Sergey M․ bff857a8af
[postprocessor/embedthumbnail] Fix issues (closes #25717)
* Fix WebP with wrong extension processing
* Fix embedding of thumbnails with % character in path
4 years ago
Alex Merkel a31a022efd
[postprocessor/embedthumbnail] Add support for non jpeg/png thumbnails (closes #25687) 4 years ago
Sergey M․ 2791e80b60
[postprocessor/ffmpeg] Embed series metadata with --add-metadata 5 years ago
Sergey M․ 824fa51165
[utils] Improve subtitles_filename (closes #22753) 5 years ago
Sergey M․ 3089bc748c
Fix W504 and disable W503 (closes #20863) 6 years ago
Sergey M․ ce52c7c111
[postprocessor/ffmpeg] Fix avconv processing broken in #19025 (closes #19067) 6 years ago
Sergey M․ 61ff92e11e
[postprocessor/ffmpeg] Wrap loglevel args in encodeArgument 6 years ago
Sergey M 1397a790ff
[postprocessor/ffmpeg] Disable "Last message repeated" messages which cause non-zero exit status (#19025) 6 years ago
Tatsh 7f903dd8bf [postprocessor/ffmpeg] Do not copy Apple TV chapter tracks while embedding subtitles (closes #19042)
Related issue: https://trac.ffmpeg.org/ticket/6016
6 years ago
Andrew Udvare 2b3afe6b0f
[postprocessor/ffmpeg] Disable "Last message repeated" messages which cause non-zero exit status 6 years ago
Remita Amine eb35b163ad [postprocessor/ffmpeg] fallback to ffmpeg/avconv for audio codec detection(closes #681) 6 years ago
Sergey M․ cbdc688c41
[postprocessor/ffmpeg] Relax ubuntu ffmpeg version regex 6 years ago
Sergey M․ 5caa531a1a
[postprocessor/ffmpeg] PEP 8 6 years ago
Remita Amine a64646e417 [postprocessor/ffmpeg] sanitize ffmpeg version for Ubuntu and Arch Linux systems(closes #18813) 6 years ago
Sergey M․ 04fb6928da
[postprocessor/ffmpeg] Embed subtitles with non-standard language codes (refs #18765) 6 years ago
Sergey M․ d4a24f4091
Prefer ffmpeg over avconv by default (closes #8622) 6 years ago
Andrew Udvare b5cbe3d652 [postprocessor/embedthumbnail] Skip embedding when there aren't any thumbnails 7 years ago
Sergey M․ 7974e289a1
[postprocessor/xattr] Clarify NO_SPACE message (#14970) 7 years ago
Sergey M․ d2ae7e24e5
[postprocessor/ffmpeg] Convert to opus using libopus (closes #14381) 7 years ago
Yen Chi Hsuan 3869028ffb [utils] Use bytes-like objects in dfxp2srt
This fixes handling of non-UTF8 TTML subtitles

Closes #14191
7 years ago
Sergey M․ a02682fd13
Keep in sync with ffmpeg's current malformed AAC bitstream wording (closes #13587) 7 years ago
Sergey M․ 8b6ac49ecc
[postprocessor/execafterdownload] Encode command line (closes #13407) 7 years ago
Sergey M․ be80986ed9
[postprocessor/metadatafromtitle] Fix missing optional meta fields (closes #13408) 7 years ago
Yen Chi Hsuan 5192ee17e7
[postprocessor/ffmpeg] Fix metadata filename handling on Python 2
Fixes #13182
8 years ago
Sergey M․ fa26734e07
[postprocessor/metadatafromtitle] Add support regex syntax for --metadata-from-title (closes #13065) 8 years ago
remitamine fa2a36d9bc [ffmpeg] add support for chapters field postprocessing 8 years ago
Remita Amine 40fcba5edb improve coding style 8 years ago
Philipp Hagemeister 21bfcd3d6e [postprocessor/ffmpeg] Add support for flac
Requested at http://stackoverflow.com/q/42828041/35070
8 years ago
Sergey M․ 0f57447de7
[postprocessor/ffmpeg] Add mising space (closes #12232) 8 years ago
Yen Chi Hsuan ec85ded83c
Fix "invalid escape sequences" error on Python 3.6 8 years ago
Yen Chi Hsuan 8e2915d70b
Revert "[postprocessor/embedthumbnail] Allow mkv to embed thumbnails"
This reverts commit 7360db05b4.

This commit was added as an attempt to fix #6046. Unfortunately, the fix
is completely wrong. As reported on #10359, embedded thumbnails are not
displayed in VLC, and Se7en on IRC reports that the embedded thumbnail
misleads mpv as well.

The correct way is using -attachment of ffmpeg, while the current
run_ffmpeg_multiple_files API can't handle it cleanly.
8 years ago
Sergey M․ 0b94dbb115
[postprocessor/ffmpeg] PEP 8 8 years ago
Pierre Mdawar 2273e2c530 [postprocessor/ffmpeg] Return correct filepath and ext in updated information in FFmpegExtractAudioPP
Return correct audio's filepath and ext instead of the video's when extracting audio and audio file already exists.
8 years ago
Déstin Reed dcdb292fdd
Unify coding cookie 8 years ago
Yen Chi Hsuan efa97bdcf1
Move write_xattr to utils.py
There are some other places that use xattr functions. It's better to
move it to a common place so that others can use it.
8 years ago
Remita Amine 30d9e20938 [postprocessor/ffmpeg] apply FFmpegFixupM3u8PP only for videos with aac codec(#5591) 8 years ago
Sergey M․ 25dd58ca6a
[metadatafromtitle] Remove unused exception class 8 years ago
nyorain 5e42f8a0ad
Make --metadata-from-title non fatal
Output a warning if the metadata can't be parsed from the title (and don't write any metadata) instead of raising a critical error.
8 years ago
Yen Chi Hsuan 23495d6a39
Revert "[ffmpeg] Fix embedding subtitles (#9063)"
This reverts commit ccff2c404d.

Fixes #10081.

The new approach breaks embedding subtitles into video-only or
audio-only files. FFMpeg provides a trick: add '?' after the argument of
'-map' so that a missing stream is ignored. For example:

opts = [
    '-map', '0:v?',
    '-c:v', 'copy',
    '-map', '0:a?',
    '-c:a', 'copy',
    # other options...
]

Unfortunately, such a format is not implemented in avconv, either.
I guess adding '-ignore_unknown' if self.basename == 'ffmpeg' is the
best solution. However, the example mentioned in #9063 no longer serves
problematic files, so I can't test it. I'll reopen #9063 and wait for
another example so that I can test '-ignore_unknown'.
8 years ago
Yen Chi Hsuan 5b6a74856b Merge pull request #9288 from reyyed/issue#9063fix
[ffmpeg] Fix embedding subtitles (#9063)
8 years ago
Yen Chi Hsuan 702ccf2dc0
[compat] Rename shlex_quote and remove unused subprocess_check_output 9 years ago
Sergey M․ 4bd143a3a0
[postprocessor/ffmpeg] Simplify metadata preparation and add track related metafields (Closes #9357) 9 years ago
Wang Jun Tham ccff2c404d [ffmpeg] Fix embedding subtitles (#9063)
Changed command line parameters for ffmpeg when embedding subtitles.
Changed to ‘-map 0:v -c:v copy -map 0:a -c:a copy’
9 years ago
remitamine b9f2fdd37f [ffmpeg] Clarify rationale for pipe(-) exclusion in _ffmpeg_filename_argument 9 years ago