Two small CVS commits and a question

I've performed the following two small CVS commits:

1) Updated the default Internet radio stations list in
   emms-streaming.el. Cleaned out dead servers, updated server IPs and
   added a couple.

2) Changed emms-stream-info.el to use `make-network-process' instead of
   `open-network-stream', since RMS talked about reducing the amount of
   `open-network-*' procedures. `make-network-process' is not slated for
   change.

Now for the query:

The EMMS README file states that setup looks like this:

...
(require 'emms)
(require 'emms-player-simple)
(require 'emms-source-file)
(setq emms-player-list '(emms-player-mpg321
                         emms-player-ogg123
                         emms-player-mplayer)
      emms-source-list '((emms-directory-tree "~/media/")))
...

When the web-site says:

...
(require 'emms-default)

Re: Two small CVS commits and a question

Yoni Rabkin Katzenell <yoni-r <at> actcom.com> writes:

> 1) Updated the default Internet radio stations list in
> emms-streaming.el. Cleaned out dead servers, updated server IPs and
> added a couple.

Thanks !

[...]

> This should probably be changed (I can even do it myself) if this is
> OK with the grand-united-all-high-counsel-of-emms-developers.

I think that the best way to configure EMMS is the `emms-default' way,
so I'd vote for changing the README to reflect the website.

--

-- 
Lucas

Stream playlist support

Hello everybody !

EMMS now supports stream playlists, ie URLs like : 

http://www.somafm.com/tagstrance.pls

I added a special type for that, called playlist.

A variable has changed, `emms-stream-default-action', be sure to change
it. It should be "add" or "play", and no longer 'emms-add-url or
'emms-play-url.

Code is more or less tested, if you experiment bugs, be sure to
complain.

--

-- 
Lucas

Modeline icon support !

Hello !

Thanks to Daniel Brockman, EMMS now supports the display of an icon in
the modeline.

Useful variables to set : `emms-modeline-icon-color' and
`emms-modeline-icon-before-format'. Quoting the docstring :

>String to put before the icon, in the modeline.
>For example, if you want to have something like :
>[ <icon> Foo - The Foo Song ]
>You should set it to "[", and set emms-modeline-format to "%s ]"

--

-- 
Lucas

Miscellaneous patches

This avoids warnings the first time you compile emms-auto.el:

When you type M-x emms-play-directory RET /media/some/album RET, the
first song immediately starts playing --- before its info has had a
chance to load.  This results in the mode line displaying an ugly file
name for the first file, but a nice-looking title for the others.
This simple fix just loads the info for the first file synchronously:

These just fix a couple of dependencies:

This automatically sorts all tracks you add to the playlist:

--

-- 
Daniel Brockman <daniel <at> brockman.se>

`modeline' vs. `mode-line'

I think it is important to use consistent wording (for practical as
well as aesthetical reasons) and since EMMS is a GNU project, I think
it is reasonable to expect it to adhere to GNU Emacs conventions.

Now, people have spent effort on changing every mention of `modeline'
to `mode-line' in GNU Emacs --- deprecating variables, faces, etc.
Could it be that this is something that should be considered for EMMS?
Well, except the part about deprecating:  We could just change, right?

It'll be easier to do this before total world domination is achieved.
I volunteer to do the actual work if people agree that it would be a
reasonable change; I have some stuff I need to procrastinate anyway,
and technically pointless changes like this turn me on.

--

-- 
Daniel Brockman <daniel <at> brockman.se>

Patch for emms-streaming.el

Below is a patch for emms-streaming.el, I hope gmail will not munge it.

The radio stations have been updated to use Lucas' playlist type.
"Deephouse" and "Mostly Classical" from DigitallyImported still do not
work, but according to the station's Website this might be because of
technical errors. All other stations have been checked to work.

I've removed `emms-stream-info-bookmark' and replaced it with a
placeholder function `emms-stream-info-current' since streaming
information if currently broken until I get it to work with the new
changes.

START OF DIFF OUTPUT
*** emms/emms-streaming.el	2005-05-18 17:11:22.000000000 +0300
--- modified_emms/emms-streaming.el	2005-05-19 09:37:12.000000000 +0300
***************
*** 60,95 ****
  ;; type could be either url or playlist. If url, then it represents a
  ;; direct IP, otherwite it's a stream playlist
  (defvar emms-stream-default-list
!   '(("SomaFM: Beatblender" "http://64.124.44.102:8388" 1 url)
!     ("SomaFM: Secret Agent" "http://207.200.96.225:8010" 1 url)
!     ("SomaFM: Secret Agent" "http://64.236.34.67:80/stream/1021" 2 url)
!     ("SomaFM: Groove Salad" "http://64.236.34.97:80/stream/1018" 1 url)
!     ("SomaFM: Groove Salad" "http://64.236.34.196:80/stream/1018" 2 url)
!     ("SomaFM: Drone Zone" "http://64.236.34.97:80/stream/1032" 1 url)
!     ("SomaFM: Drone Zone" "http://server2.somafm.com:8002" 2 url)
!     ("SomaFM: Tag's Trance" "http://www.somafm.com/tagstrance.pls" 1 playlist)
!     ("Digitally Imported, Deephouse"
"http://64.236.34.4:80/stream/1007" 1 url)

Patch for README

This is my patch for the README file. It is based on the Web site version.

START OF CVS OUTPUT
*** emms/README	2003-12-23 21:03:39.000000000 +0200
--- d_emms/README	2005-05-19 11:53:37.000000000 +0300
***************
*** 1,3 ****
--- 1,7 ----
+ EMMS --- The Emacs Multi-Media System
+ =====================================
+ 
+ 
  Introduction
  ============
  EMMS is the Emacs Multi-Media System. It tries to be a clean and small
***************
*** 9,15 ****

  Installation
  ============
! You need to put all the .el files of emms in a directory in your
  load-path. For example, if you put all those files into ~/elisp/emms/,
  then in your ~/.emacs, you should do:

--- 13,19 ----

  Installation
  ============
! You need to put all the .el files of EMMS in a directory in your
  load-path. For example, if you put all those files into ~/elisp/emms/,

Re: `modeline' vs. `mode-line'

Daniel Brockman <daniel <at> brockman.se> writes:

> I think it is important to use consistent wording (for practical as
> well as aesthetical reasons) and since EMMS is a GNU project, I think
> it is reasonable to expect it to adhere to GNU Emacs conventions.

Well, EMMS isn't really a GNU project. Last time I asked about papers to
sign, I was answered that EMMS didn't go through all the required steps.

But I don't mind changing that, let's see what our Big Boss thinks.

> It'll be easier to do this before total world domination is achieved.
> I volunteer to do the actual work if people agree that it would be a
> reasonable change; I have some stuff I need to procrastinate anyway,
> and technically pointless changes like this turn me on.

That's a change I can do, as soon as Jorgen agreed on it :)

--

-- 
Lucas

Re: Patch for emms-streaming.el

Yoni Rabkin <yonirabkin <at> gmail.com> writes:

> Below is a patch for emms-streaming.el, I hope gmail will not munge it.
>
> The radio stations have been updated to use Lucas' playlist type.
> "Deephouse" and "Mostly Classical" from DigitallyImported still do not
> work, but according to the station's Website this might be because of
> technical errors. All other stations have been checked to work.
>
> I've removed `emms-stream-info-bookmark' and replaced it with a
> placeholder function `emms-stream-info-current' since streaming
> information if currently broken until I get it to work with the new
> changes.

Thanks, patch applied. Be sure to update your streams, dear EMMS users.

Regards,
--

-- 
Lucas