Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bookmarks fail to load due to loadfile syntax change #20

Open
Ilethas opened this issue Mar 20, 2024 · 2 comments
Open

Bookmarks fail to load due to loadfile syntax change #20

Ilethas opened this issue Mar 20, 2024 · 2 comments

Comments

@Ilethas
Copy link

Ilethas commented Mar 20, 2024

After recent updates in mpv, there was a change in loadfile command and bookmarks no longer load. You can see the issue mentioned here: mpv-player/mpv#13656.

You can fix the problem by replacing the following line:

mp.commandv("loadfile", parsePath(bookmark["path"]), "replace", "start="..bookmark["pos"])

with this line:

mp.commandv("loadfile", parsePath(bookmark["path"]), "replace", -1, "start="..bookmark["pos"])

I saw someone make an adjustment here to make it work with both old and new mpv versions:
image

Hope it helps! 😃

@Digita1Menace
Copy link

Fixed it thanks!

@bluetoothyamada
Copy link

THANKS!!

Fossu added a commit to Fossu/mpv-bookmarker that referenced this issue Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants