mpv crashes when loading custom scripts about mpv.js HOT 9 CLOSED

using command interface to change the property doesn't crash MPV so it might load the script.

this.mpv.command('set script', path.join(__dirname, 'mpv_commons.lua'))

so I used this script as an example but it doesn't work. this could be because of mpv build wasn't compiled with lua support. but I doubt that since i'm using mpv-dev. is there a way to find out if this command failed?

from mpv.js.

Try with MPVJS_TERMINAL=1 MPVJS_VERBOSE=1 environment variables.

from mpv.js.

OK so set script is not a command. and also for this.mpv.property('script') says Do not use script, use scripts, bug reports will be ignored. but scripts is not a property. then I tried to force a different configuration directory.

this.mpv.property('config-dir', path.join(__dirname, 'mpv', 'config'))

so that mpv would load the scripts inside __dirname/mpv/config/scripts. but this time crashed with no specific reason.

this line in the log file indicates the following situation:

  handleMPVReady (mpv) {
    this.mpv = mpv
    const observe = mpv.observe.bind(mpv);
    ['pause', 'time-pos', 'duration', 'eof-reached', 'volume', 'mute'].forEach(observe)
    this.mpv.property('volume', this.state.volume)
    this.mpv.property('mute', this.state.mute)
    this.mpv.property('config-dir', path.join(__dirname, 'mpv', 'config')) //crash no info
  }

from mpv.js.

Seems like it crashes because of:

[opengl-cb/cuda-nvdec] need OpenGL >= 2.1 or OpenGL-ES >= 3.0
[opengl-cb] Loading failed.

Not because of Lua script. Does it play some test files for you without Lua script?

from mpv.js.

It seems to be related only to hwdec though.

from mpv.js.

yes I have no problem with playing files without Lua script.

from mpv.js.

in IRC someone mentioned that --script is a CLI method that appends to the --scripts option. and since --scripts isn't a property I'm guessing that it is not exposed to libmpv.

though this quote from IRC says it is possible

<CounterPillow> you can set the "scripts" property, as far as I understand

and as for config-dir. it is also a CLI option and it is mentioned that most options can be set as runtime via properties as well. but this one isn't. and properties in general might have very different behavior from the options.

I'm leaving this open for you to decide. see if it is possible to set the scripts property on your end.

from mpv.js.

turned out load-script will do the trick but it is listed as "Possibly Subject to Change". I tried it with seek-show-position.js

this.mpv.command('load-script', path.join(__dirname, 'seek-show-position.js'))

from mpv.js.

Crash was actually caused by a bug of handling string properties, see #31. Your first example should work now.

from mpv.js.