Bastian Hussi via <[email protected]> writes: > Hi there, > > I'm running the latest version of emms on Emacs 31 (build from source) > and I have witnessed this warning when I run |emms-all|: > > |⛔ Warning (files): Missing ‘lexical-binding’ cookie in > "~/.config/emacs/var/emms/cache".| > |You can add one with ‘M-x elisp-enable-lexical-binding RET’.| > |See ‘(elisp)Selecting Lisp Dialect’ and ‘(elisp)Converting to > Lexical Binding’| > |for more information.| > > Lexical binding seems to be the default since Emacs 31, see here: > https://debbugs.gnu.org/cgi/bugreport.cgi?bug=74145. > > Would it be a quickwin to just enable lexical-binding in the cache > file, like the attached patch does? Or are there more things to > consider? I'm pretty new to emms and emacs related mailing lists.
I don't see why not. I went ahead and added this change to the main git repo: [https://cgit.git.savannah.gnu.org/cgit/emms.git/commit/?id=2c902ddbab99f55a8d1ef047a9a403d4e6f61d02]. I'll also commit a few extra changes soon which will also help toward making the compilation work smoothly on emacs 31. Thank you. -- "Cut your own wood and it will warm you twice"
