Hello Pierre,

Pierre Neidhardt <ambre...@gmail.com> writes:

[…]

> Wow, impressive work indeed.  Note that you shouuld not have added
> emms-player-simple-mpv since it's superseded by Emms 5.0.

Do you mean ‘emms-player-simple.el’ from:
--8<---------------cut here---------------start------------->8---
tar xf $(~/src/guix/pre-inst-env env GUIX_PACKAGE_PATH= guix build --no-grafts 
-S emacs-emms) emms-5.0/lisp/emms-player-simple.el
--8<---------------cut here---------------end--------------->8---

I didn't check it carefully, but as I see it's not the same because I
don't see ‘json’ mentioned anywhere.  Also ‘cloc’ tells about 1936 lines
of Lisp code in ‘emms-player-simple-mpv’.  I will try ‘emms’ 5.0 later.

> I'll take care of this in a few days unless someone is faster than me :)

OK, thank you!  But please make sure that ‘emms-player-simple.el’ from
Emms is really the same as ‘emms-player-simple-mpv’, which uses MPV IPC
and parses JSON (not just starts a ‘mpv’ process per song or video).

Oleg.

Attachment: signature.asc
Description: PGP signature

Reply via email to