all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Oleg Pykhalov <go.wigust@gmail.com>
To: Pierre Neidhardt <ambrevar@gmail.com>
Cc: guix-devel@gnu.org, Alex Kost <alezost@gmail.com>
Subject: Re: 29/88: gnu: Add emacs-let-alist.
Date: Fri, 04 May 2018 09:13:43 +0300	[thread overview]
Message-ID: <87o9hw6j8o.fsf@gmail.com> (raw)
In-Reply-To: 87zi1gilr3.fsf@gmail.com

[-- Attachment #1: Type: text/plain, Size: 1072 bytes --]

Hello Pierre,

Pierre Neidhardt <ambrevar@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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

  reply	other threads:[~2018-05-04  6:13 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20180502172100.16240.36842@vcs0.savannah.gnu.org>
     [not found] ` <20180502172109.5B497204D2@vcs0.savannah.gnu.org>
2018-05-03 19:13   ` 29/88: gnu: Add emacs-let-alist Alex Kost
2018-05-03 19:26     ` Pierre Neidhardt
2018-05-04  6:13       ` Oleg Pykhalov [this message]
2018-05-04  8:39         ` Pierre Neidhardt
2018-05-04  5:52     ` Oleg Pykhalov
2018-05-04 18:50       ` Oleg Pykhalov
2018-05-06 16:55         ` Alex Kost
2018-05-06 17:39           ` Oleg Pykhalov
2018-05-07 17:11             ` Alex Kost

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87o9hw6j8o.fsf@gmail.com \
    --to=go.wigust@gmail.com \
    --cc=alezost@gmail.com \
    --cc=ambrevar@gmail.com \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.