[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [emms-help] help submitting my patches

From: Yoni Rabkin
Subject: Re: [emms-help] help submitting my patches
Date: Sun, 19 Jun 2016 16:53:15 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.95 (gnu/linux)

Alejandro Erickson <address@hidden> writes:

> Hi,
> I wrote some extra functions for EMMS as I prepared an EMMS plugin
> for Spacemacs, and I realised I should probably offer the changes to
> EMMS first.  I don't understand how to do that though (this being my
> first emacs-related contribution).  I've read https://
> www.emacswiki.org/emacs/EMMSDevelopment
> I cloned https://github.com/emacsmirror/emms at https://github.com/
> alejandroerickson/emms and made my changes, with one or so commits
> per feature.  Your doc doesn't mention github pull requests (naïve
> question perhaps, but why?), and when I tried the format-patch
> command I got an the following error:
>  % git format-patch -M emacsmirror                                  
>                                               ~/emms (master)
> alejandro
> fatal: ambiguous argument 'emacsmirror': unknown revision or path not
> in the working tree.
> Use '--' to separate paths from revisions, like this:

Emms is hosted on Savannah, please send patches to this mailing list
against the main Savannah git repo:

> The changes include:
> - seek one minute functions (many people have long, 1hr+ tracks)

Functions like this are so trivial that they are typically added by
people in their ~/.emmsrc, but if you'd like to define some new bindings
along the lines of `emms-seek-forward' I certainly don't mind adding

> - remove defunct streams (about half of them don't work, including DI
> radio that requires premium account to stream outside their app)

That would be a very useful patch. Thank you.

> - a delete function that explicitly moves to trash, rather than
> depending on whether emacs is configured to delete by trashing or not

I assume you mean a modification to the Browser, since the playlist-mode
doesn't offer file-level deletion at all (nor should it). If I want to
modify the underlying files I hit `d' in the playlist-mode buffer and
use dired. This makes sense to me instead of duplicating dired-like
functionality in Emms.

Since I don't use the Emms Browser at all, I'm probably not the right
person to weigh in on whether this makes sense. Hopefully someone who
does use the browser will reply with a useful comment.

> - play/pause dwim, that tries really hard to play music, even if the
> playlist is empty

That isn't enough for me to understand exactly what the patch does, so I
can't comment.

> There will probably be more changes as I discover and use more
> features of emms, but I'd like to submit a patch in the interim.

Please do, and thank you for working on Emms.

> Additional question: is there testing code for emms?

Nope, we keep the main git repo fairly stable and tend to break stuff in
our own private git branches. That said, there is no problem with having
people test code in the main branch since there are also stable release
tarballs to pick from.

> Note: emms-patches rejected my email.

We've depreciated emms-patches, is it still listed somewhere? It
shouldn't be.

> Alejandro
> --
> Dr. Alejandro Erickson
> Postdoctoral Research Associate in Computer Science, Durham
> University, UK
> http://alejandroerickson.com
> _______________________________________________
> Emms-help mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/emms-help

   "Cut your own wood and it will warm you twice"

reply via email to

[Prev in Thread] Current Thread [Next in Thread]