mpvacious_linux_v0.2.lua:325: '=' expected near 'ret' about mpvacious

[tatsumoto-ren]

Thank you for your feedback. To support lua 5.1 I'm going to commit the changes.

[tatsumoto-ren]

Thank you for the bug report. It's really strange but I can't get the same error. It must be platform-specific or related to the version of lua that mpv uses. Does it work with this file?

[anonx7eb]

Thanks for the quick response! The file you sent fixes the error on startup! However, Ctrl+C to copy subs doesn't work for me and throws this error:

[new_mpvvacious]
[new_mpvvacious] stack traceback:
[new_mpvvacious]        /home/<user>/.config/mpv/scripts/new_mpvvacious.lua:169: in function 'copy_to_clipboard'
[new_mpvvacious]        /home/<user>/.config/mpv/scripts/new_mpvvacious.lua:175: in function 'fn'
[new_mpvvacious]        mp.defaults:225: in function 'fn'
[new_mpvvacious]        mp.defaults:60: in function 'handler'
[new_mpvvacious]        mp.defaults:365: in function 'handler'
[new_mpvvacious]        mp.defaults:486: in function 'call_event_handlers'
[new_mpvvacious]        mp.defaults:520: in function 'dispatch_events'
[new_mpvvacious]        mp.defaults:479: in function <mp.defaults:478>
[new_mpvvacious]        [C]: ?
[new_mpvvacious]        [C]: ?
[new_mpvvacious] Lua error: /home/<user>/.config/mpv/scripts/new_mpvvacious.lua:169: attempt to index a boolean
value

Lua Version: Lua 5.1.5
OS: Gentoo

[tatsumoto-ren]

Lua 5.1.5

This must be it. mpv by default uses lua 5.2 and I expect many things to break in older versions.
This is my attempt to rewrite that chunk with 5.1 support.

[anonx7eb]

This works! I will make sure I'm on 5.2 before reporting any problems on future versions. Thanks!