Can't get seek-to.lua to work :(

[pjotrek-b]

I've downloaded your code and saved it to ".config/mpv/scripts/seek-to.lua" - and restarted MPV (as works with the other lua scripts I have in that folder).

But when I paste the following timecode using "Ctrl+Alt+V" nothing happens, and mpv says it ain't responsible for that keyboard shortcut.
The timecode was: 00:01:03.12 - and the video was 2m20s long. So it should work, right?

Can I help debugging this?

Thank you very much in advance!

System:

• Xubuntu 20.04
• mpv v0.32.0 (I know it ain't the latest)

[dexeonify]

Hmm, that's definitely unexpected. The paste_timestamp function of the script should work OOTB. If the timestamp is invalid, or there's a problem retrieving the clipboard data, there should be an error message in the console. To be clear, this script isn't tested on Linux, but if what you say is true:

mpv says it ain't responsible for that keyboard shortcut.

then it failed at the registering the keybinds, which suggests a different issue entirely.

Does manually adding the keybinds in input.conf works? For example:

t        script-message-to seek_to toggle-seeker
Alt+t    script-message-to seek_to paste-timestamp

If neither of the keybinds work, it's probably an issue in your mpv config.

[pjotrek-b]

Thanks for your reply! :)

I've tried again now, and keybinding seems to be recognized now.
No more message:
"No key binding found for key 'Ctrl+Alt+V'".

So key-bindings seem to be fine now.
With the above lines written to (a new) "/etc/input.conf", pressing "t" shows me a "seek to" message - but "Alt+T" (Or Ctrl+Alt+V) still does nothing.

I'm starting MPV from the commandline to see the logging output in the shell:
I don't get any error messages. Just "nothing happens".

[pjotrek-b]

I've added a dummy-messages to "paste_timestamp()" - to see if (and how far) it's being executed:

Seems it never gets past the "if clipboard == nil then return end" line.
I suspect it may not be able to read the clipboard contents on my system. Will check if I can edit "get_clipboard()" to make it work 😄

[pjotrek-b]

Seems that the "platform" string cannot be read (is nil)?

local platform = mp.get_property_native("platform")
mp.osd_message("platform" .. platform)

Throws:

[seek_to] Lua error: ~/.config/mpv/scripts/seek-to.lua:8: attempt to concatenate local 'platform' (a nil value)

...so the "get_clipboard()" function has no chance 😉
I'll see if I can list MPV's properties to find out why.

[dexeonify]

Ah yes, the platform property was added in mpv-player/mpv@80feac6, which is only available from v0.36.0 forward. I've anticipated it, but I also didn't feel like adding workaround/hacks for the few people who are aware about my fork.

I would definitely vouch for upgrading mpv, but if you can't, you can either hardcode the platform variable as linux (an ugly solution, I know) or implement the boilerplate code mentioned in the mpv commit above.