search

0dist / oscc

Dark-themed interface for the mpv player
45 stars 1 forks source link

"The shared-script-properties property is deprecated and will be removed in the future. Use the user-data property instead." #6

Open pitsi opened 8 months ago

pitsi commented 8 months ago

The message in the title is what pops up at the very first line of mpv when playing any file or stream and it is due to a change in mpv 0.37. Unfortunately, I do not have the knowledge to fix it, but some other (forks of) osc scripts have already done it, e.g.

https://github.com/zydezu/ModernX/commit/8819a61618048b89bb1fc20b39458f371ea6e62a

po5 commented 8 months ago

The ModernX fork you linked drops compatibility for mpv <0.37. Use the solution from thumbfast instead: https://github.com/po5/thumbfast/commit/fb9d37e76b97a3a541004556ec19c0620ca308a3

pitsi commented 8 months ago

Do you expect me to find all those instances on a file that is not the same as the one mentioned in the commit? I can not do that, sorry. As for the compatibility, I respect what you say, but I think it would be better for the script to be ready for mpv 0.38 (assuming the forementioned "property" will be completely removed in 0.38) and have 2 versions, one for the 0.36 and older and one for 0.37 and newer. After all, everyone will probably upgrade to mpv 0.37+ in a few years.

po5 commented 8 months ago

lol for taking this personally

pitsi commented 6 months ago

I just noticed that the last commit mentions it does not support mpv 0.37 or newer. I am leaving this open though, in case the dev fixes it in the future. If not, feel free to close it. I will keep using it though and ignore the warning, until any newer version of mpv has issues with it.