Discussion:
[emms-help] [PATCH] Play tracks randomly
Daimrod
2013-05-10 10:11:21 UTC
Permalink
Hello,

I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
Yoni Rabkin
2013-05-10 14:48:19 UTC
Permalink
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
--
"Cut your own wood and it will warm you twice"
Daimrod
2013-05-10 15:39:20 UTC
Permalink
Post by Yoni Rabkin
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
I agree they're close, but this one doesn't shuffle the playlist and it
persists when the playlist is changed.
--
Daimrod/Greg
Yoni Rabkin
2013-05-10 15:45:03 UTC
Permalink
Post by Daimrod
Post by Yoni Rabkin
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
I agree they're close, but this one doesn't shuffle the playlist and it
persists when the playlist is changed.
I certainly have no problem with it; I was just making sure you were
aware of emms-shuffle.

If Lucas is fine with installing it (I don't think it needs assignment
papers) I would ask that you please also send in a patch to the manual
documenting this change.
--
"Cut your own wood and it will warm you twice"
Daimrod
2013-05-10 17:49:58 UTC
Permalink
Post by Yoni Rabkin
Post by Daimrod
Post by Yoni Rabkin
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
I agree they're close, but this one doesn't shuffle the playlist and it
persists when the playlist is changed.
I certainly have no problem with it; I was just making sure you were
aware of emms-shuffle.
If Lucas is fine with installing it (I don't think it needs assignment
papers) I would ask that you please also send in a patch to the manual
documenting this change.
I have already made a copyright assignment for Emacs and I have no
problem to do it for EMMS if it is required.

I've attached the patch with the documentation.
Lucas Bonnet
2013-05-15 09:44:02 UTC
Permalink
Post by Yoni Rabkin
Post by Daimrod
Post by Yoni Rabkin
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
I agree they're close, but this one doesn't shuffle the playlist and it
persists when the playlist is changed.
I certainly have no problem with it; I was just making sure you were
aware of emms-shuffle.
If Lucas is fine with installing it (I don't think it needs assignment
papers) I would ask that you please also send in a patch to the manual
documenting this change.
I'd say it's small enough to fly below the copyright papers radar :)

Thanks for your patch!

Regards,
--
Lucas
Daimrod
2013-05-30 12:09:26 UTC
Permalink
Post by Lucas Bonnet
Post by Yoni Rabkin
Post by Daimrod
Post by Yoni Rabkin
Post by Daimrod
Hello,
I've attached a small patch which adds `emms-toggle-random-playlist'. It
sets `emms-player-next-function' to either `emms-next-noerror' or
`emms-random'.
How is this different from running M-x emms-shuffle on the playlist?
I agree they're close, but this one doesn't shuffle the playlist and it
persists when the playlist is changed.
I certainly have no problem with it; I was just making sure you were
aware of emms-shuffle.
If Lucas is fine with installing it (I don't think it needs assignment
papers) I would ask that you please also send in a patch to the manual
documenting this change.
I'd say it's small enough to fly below the copyright papers radar :)
Thanks for your patch!
Regards,
Oops, there is a small typo in the patch, here is the fix. Sorry for the
inconvenience.
Lucas Bonnet
2013-05-30 16:20:47 UTC
Permalink
Post by Daimrod
Post by Lucas Bonnet
Thanks for your patch!
Regards,
Oops, there is a small typo in the patch, here is the fix. Sorry for the
inconvenience.
Applied :)


Regards,
--
Lucas Bonnet
Loading...