160 Commits

• [postprocessor/ffmpeg] Convert to opus using libopus (closes #14381)
  Sergey M․ · 2017-10-02 04:43:25 +07:00
  d2ae7e24e5
• [utils] Use bytes-like objects in dfxp2srt

  This fixes handling of non-UTF8 TTML subtitles
  
  Closes #14191

  Yen Chi Hsuan · 2017-09-16 12:18:38 +08:00
  3869028ffb
• Keep in sync with ffmpeg's current malformed AAC bitstream wording (closes #13587)
  Sergey M․ · 2017-07-09 17:09:44 +07:00
  a02682fd13
• [postprocessor/execafterdownload] Encode command line (closes #13407)
  Sergey M․ · 2017-06-17 23:16:53 +07:00
  8b6ac49ecc
• [postprocessor/metadatafromtitle] Fix missing optional meta fields (closes #13408)
  Sergey M․ · 2017-06-17 19:05:10 +07:00
  be80986ed9
• [postprocessor/ffmpeg] Fix metadata filename handling on Python 2

  Fixes #13182

  Yen Chi Hsuan · 2017-05-25 22:07:03 +08:00
  5192ee17e7
• [postprocessor/metadatafromtitle] Add support regex syntax for --metadata-from-title (closes #13065)
  Sergey M․ · 2017-05-14 00:03:15 +07:00
  fa26734e07
• [ffmpeg] add support for chapters field postprocessing
  remitamine · 2017-05-02 20:41:48 +01:00
  fa2a36d9bc
• improve coding style
  Remita Amine · 2017-04-12 20:38:43 +01:00
  40fcba5edb
• [postprocessor/ffmpeg] Add support for flac

  Requested at http://stackoverflow.com/q/42828041/35070

  Philipp Hagemeister · 2017-03-16 12:50:45 +01:00
  21bfcd3d6e
• [postprocessor/ffmpeg] Add mising space (closes #12232)
  Sergey M․ · 2017-02-24 04:56:58 +07:00
  0f57447de7
• Fix "invalid escape sequences" error on Python 3.6
  Yen Chi Hsuan · 2017-01-06 00:58:56 +08:00
  ec85ded83c
• 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.

  Yen Chi Hsuan · 2016-10-20 15:07:19 +08:00
  8e2915d70b
• [postprocessor/ffmpeg] PEP 8
  Sergey M․ · 2016-10-16 19:16:47 +07:00
  0b94dbb115
• [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.

  Pierre Mdawar · 2016-10-16 02:12:03 +07:00
  2273e2c530
• Unify coding cookie
  Déstin Reed · 2016-10-03 23:44:29 +07:00
  dcdb292fdd
• 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.

  Yen Chi Hsuan · 2016-09-30 00:28:32 +08:00
  efa97bdcf1
• [postprocessor/ffmpeg] apply FFmpegFixupM3u8PP only for videos with aac codec(#5591)
  Remita Amine · 2016-09-16 22:06:55 +01:00
  30d9e20938
• [metadatafromtitle] Remove unused exception class
  Sergey M․ · 2016-08-09 04:01:05 +07:00
  25dd58ca6a
• 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.

  nyorain · 2016-08-09 03:56:22 +07:00
  5e42f8a0ad
• 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'.

  Yen Chi Hsuan · 2016-07-15 20:02:36 +08:00
  23495d6a39
• Merge pull request #9288 from reyyed/issue#9063fix

  [ffmpeg] Fix embedding subtitles (#9063)

  Yen Chi Hsuan · 2016-07-09 14:29:53 +08:00
  5b6a74856b
• [compat] Rename shlex_quote and remove unused subprocess_check_output
  Yen Chi Hsuan · 2016-05-10 16:00:21 +08:00
  702ccf2dc0
• [postprocessor/ffmpeg] Simplify metadata preparation and add track related metafields (Closes #9357)
  Sergey M․ · 2016-05-01 10:56:54 +06:00
  4bd143a3a0
• [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’

  Wang Jun Tham · 2016-04-24 00:08:02 +08:00
  ccff2c404d
• [ffmpeg] Clarify rationale for pipe(-) exclusion in _ffmpeg_filename_argument
  remitamine · 2016-04-16 21:50:13 +01:00
  b9f2fdd37f
• [ffmpeg] check for - file name in _ffmpeg_filename_argument
  remitamine · 2016-04-16 19:45:56 +01:00
  d868f43c58
• [ffmpeg] try to convert tt subtitles usng dfxp2srt
  remitamine · 2016-04-01 19:47:49 +01:00
  0750b2491f
• [postprocessort/ffmpeg] Allow embedding webvtt into webm (Closes #8874)
  Sergey M․ · 2016-03-20 04:12:34 +06:00
  40025ee2a3
• Merge pull request #8611 from remitamine/ffmpegfd

  [downloader/external] Add FFmpegFD

  remitamine · 2016-03-13 21:30:27 +01:00
  2e7e561c1d
• [ffmpeg] convert format ext to ffmpeg output formats codes
  remitamine · 2016-03-13 12:15:29 +01:00
  a755f82549
• Merge pull request #8718 from remitamine/m3u8-fixup

  Add fixup for media files produced by HlsNative downloader(fixes #4776)

  remitamine · 2016-03-05 18:37:28 +01:00
  090eb8e25f
• [jython] Introduce compat_os_name

  os.name is always 'java' on Jython

  Yen Chi Hsuan · 2016-03-03 19:24:24 +08:00
  e9c0cdd389
• Add fixup for media files produced by HlsNative downloader(fixes #4776)
  remitamine · 2016-03-01 21:10:41 +01:00
  f17f86513e
• [postprocessor/embedthumbnail] Allow mkv to embed thumbnails

  Fixes #6046

  Yen Chi Hsuan · 2016-02-21 03:32:03 +08:00
  7360db05b4
• [refactor] Single quotes consistency
  Sergey M․ · 2016-02-14 15:37:17 +06:00
  611c1dd96e
• [FFmpegSubtitlesConvertorPP] remove unused variable
  Jaime Marquínez Ferrándiz · 2016-02-06 19:04:53 +01:00
  d4f8e83404
• [FFmpegSubtitlesConvertorPP] remove intermediate srt files
  Jaime Marquínez Ferrándiz · 2016-02-06 19:04:18 +01:00
  7b8b007cd9
• [FFmpegSubtitlesConvertorPP] correctly update the extension (fixes #8444)
  Jaime Marquínez Ferrándiz · 2016-02-06 18:58:18 +01:00
  3547d26587
• [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.

  Jaime Marquínez Ferrándiz · 2016-02-06 18:51:05 +01:00
  7e62c2eb6d
• [ffmpeg] check for m3u8 protocol in FFmpegMetadataPP
  remitamine · 2016-02-05 17:12:49 +01:00
  7d3a035ee0
• [ffmpeg] fix condition(fixes #8440)
  remitamine · 2016-02-05 15:12:06 +01:00
  593e0f43b4
• [ffmpeg] fix adding metadata when using --hls-prefer-native(#8350)
  remitamine · 2016-02-02 22:14:23 +01:00
  6fba62c87a
• Merge pull request #8354 from remitamine/m3u8_metadata

  [ffmpeg] fix adding metadata when using m3u8_native(fixes #8350)

  remitamine · 2016-02-02 19:13:58 +01:00
  fae45ede08
• [FFmpegSubtitlesConvertorPP] delete old subtitle files (fixes #8382)
  Jaime Marquínez Ferrándiz · 2016-01-31 14:22:36 +01:00
  e04398e397
• [ffmpeg] fix adding metadata when using m3u8_native(fixes #8350)
  remitamine · 2016-01-28 18:57:32 +01:00
  cf57433bbd
• [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`.

  Juan M Martínez · 2015-12-09 20:56:00 -03:00
  374c761e77
• [postprocessor/ffmpeg] Use ffmpeg as prefix since it's used all over the places (Closes #7371)
  Sergey M․ · 2015-11-06 21:56:31 +06:00
  deb85c32bb
• [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 '-'.

  Jaime Marquínez Ferrándiz · 2015-09-18 14:38:35 +02:00
  8a7bbd1606
• [postprocessor/common] Use 'self._downloader.params' instead of 'self.params'

  'self.params' is not defined

  Jaime Marquínez Ferrándiz · 2015-09-05 12:11:36 +02:00
  3026164b16