RepoMirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv synced 2025-03-22 11:18:32 +00:00
Code Issues Releases Wiki Activity

client API: remove ytdl=no default

Browse Source 
With the recent changes to the script it does not incur a startup delay
by default due to starting youtube-dl and waiting for it. This was the
main reason for making libmpv have a different default.

Starting sub processes from a library can still be a bit fishy, but I
think it's ok. Still mention it in the libmpv header. There were already
other cases where libmpv would start its own processes, such as the X11
backend calling xdg-screensaver. (The reason why this is fishy is
because UNIX process management sucks: SIGCHLD and the wait() syscall
make sub processes non-transparent and could potentially introduce
conflicts with code trying to use them.)
This commit is contained in:
wm4 2018-01-11 23:36:07 +01:00 committed by Kevin Mitchell
parent 55993fde6d
commit 69d062ce37
4 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
DOCS/interface-changes.rst
View File

@ -24,6 +24,7 @@ Interface changes
--ad-spdif-dtshd, --softvol options --ad-spdif-dtshd, --softvol options
- fix --external-files: strictly never select any tracks from them, unless - fix --external-files: strictly never select any tracks from them, unless
explicitly selected (this may or may not be expected) explicitly selected (this may or may not be expected)
- --ytdl is now always enabled, even for libmpv
--- mpv 0.28.0 --- --- mpv 0.28.0 ---
- rename --hwdec=mediacodec option to mediacodec-copy, to reflect - rename --hwdec=mediacodec option to mediacodec-copy, to reflect
conventions followed by other hardware video decoding APIs conventions followed by other hardware video decoding APIs

3
DOCS/man/options.rst
View File

@ -547,8 +547,7 @@ Program Behavior
Enable the youtube-dl hook-script. It will look at the input URL, and will Enable the youtube-dl hook-script. It will look at the input URL, and will
play the video located on the website. This works with many streaming sites, play the video located on the website. This works with many streaming sites,
not just the one that the script is named after. This requires a recent not just the one that the script is named after. This requires a recent
version of youtube-dl to be installed on the system. (Enabled by default, version of youtube-dl to be installed on the system. (Enabled by default.)
except when the client API / libmpv is used.)
If the script can't do anything with an URL, it will do nothing. If the script can't do anything with an URL, it will do nothing.

1
etc/builtin.conf
View File

@ -17,7 +17,6 @@ idle=yes
terminal=no terminal=no
input-terminal=no input-terminal=no
osc=no osc=no
ytdl=no
input-default-bindings=no input-default-bindings=no
input-vo-keyboard=no input-vo-keyboard=no
# OSX/Cocoa global input hooks # OSX/Cocoa global input hooks

2
libmpv/client.h
View File

@ -134,6 +134,8 @@ extern "C" {
* - The FPU precision must be set at least to double precision. * - The FPU precision must be set at least to double precision.
* - On Windows, mpv will call timeBeginPeriod(1). * - On Windows, mpv will call timeBeginPeriod(1).
* - On memory exhaustion, mpv will kill the process. * - On memory exhaustion, mpv will kill the process.
* - In certain cases, mpv may start sub processes (such as with the ytdl
* wrapper script).
* *
* Encoding of filenames * Encoding of filenames
* --------------------- * ---------------------