mpv-scripts icon indicating copy to clipboard operation
mpv-scripts copied to clipboard

SimpleHistory.lua not backwards compatible with old mpvHistory.log

Open arscynic opened this issue 3 years ago • 1 comments

When replacing the old smarthistory.lua with the new SimpleHistory.lua, the new version won't work if the old mpvHistory.log is still located in /.config/mpv/. It doesn't even throw an error message.

Moreover, because my old version mentions "Version: 1.7.1" and the new version mentions "Version: 1.1", I incorrectly assumed that the new one was a deprecated old version.

arscynic avatar Sep 10 '22 17:09 arscynic

Sorry for that. Backward incompatibility is mentioned in the 2.0 release.

Writing backwards compatibility code will result in some unnecessary checks, so I decided to drop it.

I'll need to check why I didn't add an error message though, will keep issue open until then..

Eisa01 avatar Sep 10 '22 23:09 Eisa01