Remove emms-info-ogg.el/ogg-comment.el?

Hi,

since some time ago the manual recommends using
emms-info-*ogginfo*.el to retrieve Ogg Vorbis comments. The
only major advantage of emms-info-*ogg*.el, not to have to
rely on external programs, is gone since nowadays it uses
ogginfo to retrieve the playback length as well. Further,
the native parser seems to fail on non-ASCII comments:

| ELISP> (oggc-read-header "/usr/share/jukebox/Björk, David Arnold: Play Dead.ogg")
| ("Xiph.Org libVorbis I 20020717"
|  (("artist" "Bj\303\266rk, David Arnold")
|   ("title" "Play Dead")))

Therefore, rather than spending time to fix this, I propose
to remove emms-info-*ogg*.el and ogg-comment.el completely.

Tim

Re: emms-lastfm-client.el: quite a few byte compile warnings

Yoni Rabkin <yoni <at> rabkins.net> writes:

> William Xu <william.xwl <at> gmail.com> writes:
>
>> Should be fixed? 
>
> I'll make sure to fix all of those and any new ones introduced when I
> push out the new version of emms-lastfm-client.el.

I've just pushed out the new version of emms-lastfm-client.el and the
accompanying manual update (no compilation errors).

--

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

Enhancing emms-mode-line.el

Hi,

I'm thinking about enhancing emms-mode-line.el (and inte-
grating emms-mode-line-icon.el in the process).

  Instead of the present, "take it or write your own func-
tion" approach, I would like to use something like
battery-mode-line-format with a bunch of "%a" -> track's
artist, "%t" -> track's title, "%s" -> old format, "%i" ->
icon mappings.

  The problem I see when looking at - hmmm -
emms-track-simple-description is that the format should de-
pend on the track type. This could be dealt with by an
alist (always as an alternative to a personal function):

| '((file . "%i %a: %t")
|   (url  . "%i %u")
|   (t    . "%T: %N"))

Are there scenarios that don't fit well into this?

Tim

Re: Remove emms-info-ogg.el/ogg-comment.el?

Tim Landscheidt <tim <at> tim-landscheidt.de> writes:

> Hi,
>
> since some time ago the manual recommends using
> emms-info-*ogginfo*.el to retrieve Ogg Vorbis comments. The
> only major advantage of emms-info-*ogg*.el, not to have to
> rely on external programs, is gone since nowadays it uses
> ogginfo to retrieve the playback length as well. Further,
> the native parser seems to fail on non-ASCII comments:
>
> | ELISP> (oggc-read-header "/usr/share/jukebox/Björk, David Arnold: Play Dead.ogg")
> | ("Xiph.Org libVorbis I 20020717"
> |  (("artist" "Bj\303\266rk, David Arnold")
> |   ("title" "Play Dead")))

Same here.

> Therefore, rather than spending time to fix this, I propose
> to remove emms-info-*ogg*.el and ogg-comment.el completely.

Makes sense, since there are: -ogginfo and -libtag that are both able to
read Ogg comments. I'll remove it.

Regards,
--

-- 
Lucas

Tim Landscheidt <tim <at> tim-landscheidt.de> writes:

Re: Enhancing emms-mode-line.el

Tim Landscheidt <tim <at> tim-landscheidt.de> writes:

   Hi,

   I'm thinking about enhancing emms-mode-line.el (and inte-
   grating emms-mode-line-icon.el in the process).

     Instead of the present, "take it or write your own func-
   tion" approach, I would like to use something like
   battery-mode-line-format with a bunch of "%a" -> track's
   artist, "%t" -> track's title, "%s" -> old format, "%i" ->
   icon mappings.

     The problem I see when looking at - hmmm -
   emms-track-simple-description is that the format should de-
   pend on the track type. This could be dealt with by an
   alist (always as an alternative to a personal function):

   | '((file . "%i %a: %t")
   |   (url  . "%i %u")
   |   (t    . "%T: %N"))

This would be a nice feature to have!  I hope it would also support
width arguement before those format control strings, just like `format'.
e.g., "%10T".  

--

-- 
William

http://xwl.appspot.com

Re: Re: EMMS new release?

Hi,

While waiting for a new release  of EMMS, I have uploaded a git snapshot
in  Debian (experimental).   Thus, in  case of  there is  no  new stable
release of EMMS before the freeze  for the next version of Debian, I can
still put this git snapshot instead of an old 3.0.

I have  attached two  patches fixing the  Makefile (the second  one also
removes deb-install  rule which is  not used anymore and  thus useless).
Could you please push them if you think they are correct? Thanks much!

Regards,
--
Arnaud Fontaine (arnau)

Hi,

While waiting for a new release  of EMMS, I have uploaded a git snapshot
in  Debian (experimental).   Thus, in  case of  there is  no  new stable
release of EMMS before the freeze  for the next version of Debian, I can
still put this git snapshot instead of an old 3.0.

I have  attached two  patches fixing the  Makefile (the second  one also
removes deb-install  rule which is  not used anymore and  thus useless).
Could you please push them if you think they are correct? Thanks much!