40fcba5edb
improve coding style
2017-04-12 20:38:43 +01:00
21bfcd3d6e
[postprocessor/ffmpeg] Add support for flac
...
Requested at http://stackoverflow.com/q/42828041/35070
2017-03-16 12:50:45 +01:00
0f57447de7
[postprocessor/ffmpeg] Add mising space ( closes #12232 )
2017-02-24 04:56:58 +07:00
0b94dbb115
[postprocessor/ffmpeg] PEP 8
2016-10-16 19:16:47 +07:00
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.
2016-10-16 02:12:03 +07:00
30d9e20938
[postprocessor/ffmpeg] apply FFmpegFixupM3u8PP only for videos with aac codec( #5591 )
2016-09-16 22:06:55 +01:00
23495d6a39
Revert "[ffmpeg] Fix embedding subtitles ( #9063 )"
...
This reverts commit ccff2c404dFixes #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'.
2016-07-15 20:02:36 +08:00
5b6a74856b
Merge pull request #9288 from reyyed/issue#9063fix
...
[ffmpeg] Fix embedding subtitles (#9063 )
2016-07-09 14:29:53 +08:00
4bd143a3a0
[postprocessor/ffmpeg] Simplify metadata preparation and add track related metafields ( Closes #9357 )
2016-05-01 10:56:54 +06:00
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’
2016-04-24 00:08:02 +08:00
b9f2fdd37f
[ffmpeg] Clarify rationale for pipe(-) exclusion in _ffmpeg_filename_argument
2016-04-16 21:50:13 +01:00
d868f43c58
[ffmpeg] check for - file name in _ffmpeg_filename_argument
2016-04-16 19:45:56 +01:00
0750b2491f
[ffmpeg] try to convert tt subtitles usng dfxp2srt
2016-04-01 19:47:49 +01:00
40025ee2a3
[postprocessort/ffmpeg] Allow embedding webvtt into webm ( Closes #8874 )
2016-03-20 04:12:34 +06:00
2e7e561c1d
Merge pull request #8611 from remitamine/ffmpegfd
...
[downloader/external] Add FFmpegFD
2016-03-13 21:30:27 +01:00
a755f82549
[ffmpeg] convert format ext to ffmpeg output formats codes
2016-03-13 12:15:29 +01:00
f17f86513e
Add fixup for media files produced by HlsNative downloader( fixes #4776 )
2016-03-01 21:10:41 +01:00
d4f8e83404
[FFmpegSubtitlesConvertorPP] remove unused variable
2016-02-06 19:04:53 +01:00
7b8b007cd9
[FFmpegSubtitlesConvertorPP] remove intermediate srt files
2016-02-06 19:04:18 +01:00
3547d26587
[FFmpegSubtitlesConvertorPP] correctly update the extension ( fixes #8444 )
2016-02-06 18:58:18 +01:00
7e62c2eb6d
[FFmpegSubtitlesConvertorPP] fix not working when srt is used as the intermediate format between ttml/dfxp and other format
...
It was trying to use the ttml/dfxp file with ffmpeg, which doesn't have support for them.
I broke it in e04398e397
2016-02-06 18:51:05 +01:00
7d3a035ee0
[ffmpeg] check for m3u8 protocol in FFmpegMetadataPP
2016-02-05 17:12:49 +01:00
593e0f43b4
[ffmpeg] fix condition( fixes #8440 )
2016-02-05 15:12:06 +01:00
6fba62c87a
[ffmpeg] fix adding metadata when using --hls-prefer-native( #8350 )
2016-02-02 22:14:23 +01:00
fae45ede08
Merge pull request #8354 from remitamine/m3u8_metadata
...
[ffmpeg] fix adding metadata when using m3u8_native(fixes #8350 )
2016-02-02 19:13:58 +01:00
e04398e397
[FFmpegSubtitlesConvertorPP] delete old subtitle files ( fixes #8382 )
2016-01-31 14:22:36 +01:00
cf57433bbd
[ffmpeg] fix adding metadata when using m3u8_native( fixes #8350 )
2016-01-28 18:57:32 +01:00
374c761e77
[FFmpegPostProcessor] Default of prefer ffmpeg
...
When no `downloader` is passed to `FFmpegPostProcessor`
an exception was raised trying to get the prefer ffmpeg param.
AttributeError: 'NoneType' object has no attribute 'params'
This fixes and defaults to `False`.
2015-12-09 20:56:00 -03:00
deb85c32bb
[postprocessor/ffmpeg] Use ffmpeg as prefix since it's used all over the places ( Closes #7371 )
2015-11-06 21:56:31 +06:00
8a7bbd1606
[postprocessor/ffmpeg] Always use the 'file:' protocol for filenames ( fixes #6874 )
...
If the filename contains ':' it is interpreted as a protocol.
It also handles filenames starting with '-'.
2015-09-18 14:38:35 +02:00
9750e7d70e
[postprocessor/ffmpeg] Don't use '[youtube] ...' in messages
...
Because it can be used for other extractors.
2015-07-14 12:56:32 +02:00
15006fedb9
[postprocessor/ffmpeg] Spread postprocessor args usage on all ffmpeg extractors
2015-07-11 22:42:03 +06:00
f72b0a6032
Revert xvid to avi and make docs to be similar to existing external downloader option
2015-07-11 22:15:16 +06:00
ac9ed061ec
Merge branch 'master' of https://github.com/aurium/youtube-dl into aurium-master
2015-07-11 21:58:12 +06:00
aa5d9a79d6
Simplify `postprocessor_args` transmission to PP base class
...
* Remove `extra_cmd_args` transmission from sub to super class.
* Simplify params transmission through `downloader.params`.
2015-07-01 20:12:26 -03:00
1866432db7
Rename --pp-params to --postprocessor-args and access value as super class attribute
2015-06-30 16:22:09 -03:00
396726244a
[utils/ffmpeg] Move ISO 639 related codes to utils
2015-06-21 18:53:17 +08:00
d84f1d14b5
Adds support for XviD output with extra parametrization
...
As the "LG Time Machine" (a (not so) smart TV) has a limitation for video dimensions (as for codecs), I take to implement an extra parameter `--pp-params` where we can send extra parameterization for the video converter (post-processor).
Example:
```
$ youtube-dl --recode-video=xvid --pp-params='-s 720x480' -c https://www.youtube.com/watch?v=BE7Qoe2ZiXE
```
That works fine on a 4yo LG Time Machine.
Closes #5733
2015-06-09 22:08:16 -03:00
13763ce599
[postprocessor/ffmpeg] Add `can_merge` method
2015-05-11 02:00:31 +06:00
70484b9f8a
[postprocessor/ffmpeg] Extract `check_outdated` method
2015-05-11 00:26:39 +06:00
083c1bb960
Add ability to embed subtitles in mkv files ( closes #5434 )
2015-05-01 11:54:40 +02:00
bf6427d2fb
[ffmpeg] Add dfxp (TTML) subtitles support ( #3432 , #5146 )
2015-04-25 23:18:27 +08:00
c0ea8ebb9b
[ffmpeg] Remove unneeded class
2015-04-24 09:11:39 +02:00
ddbed36455
[embedthumbnail] Add support for mp3 cover embedding
2015-04-24 08:48:49 +02:00
5b5fbc0867
Detect already merged videos
...
Without the '--keep-video' option the two files would be downloaded again and even using the option, ffmpeg would be run again, which for some videos can take a long time.
We use a temporary file with ffmpeg so that the final file only exists if it success
2015-04-19 17:51:41 +02:00
d47aeb2252
FFmpegMergerPP: use the new system for specifying which files can be delete
2015-04-18 11:52:36 +02:00
14523ed969
FFmpegEmbedSubtitlePP: remove the subtitle files if '--keep-video' is not given ( closes #5435 )
2015-04-18 11:44:42 +02:00
592e97e855
Postprocessors: use a list for the files that can be deleted
...
We could only know if we had to delete the original file, but this system allows to specify us more files (like subtitles).
2015-04-18 11:36:42 +02:00
ce81b1411d
FFmpegExtractAudioPP: Simplify handling of already existing files
2015-04-17 22:37:27 +02:00
dd29eb7f81
[postprocessor/common:postprocessor/ffmpeg] Generalize utime
2015-04-08 21:40:31 +06:00
Remita Amine
Philipp Hagemeister
Sergey M․
Pierre Mdawar
Yen Chi Hsuan
Wang Jun Tham
Jaime Marquínez Ferrándiz
Juan M Martínez
Aurélio A. Heckert
Nikoli
pulpe