The"new" bits of transient that prevent compilation on 29.3 are

* the documentation block at the top of the form
* the toggling of advanced search

I will have a look and see if I can replicate these with an older version
of transient.  Otherwise I guess we wait until emacs-30 becomes
wide-spread...

---Fran



On Tue, 14 Jan 2025 at 00:18, Yoni Rabkin <[email protected]> wrote:

>
> "Fran Burstall (Gmail)" <[email protected]> writes:
>
> > But I can reproduce with emacs 29.3.  How frustrating: it seems that
> > transient has changed quite a bit in the last year or two.  I will
> > experiment...
> >
> > ---Fran
>
> I was on the way to saying the same thing and you beat me to it. We'll
> need it to work on the latest released version of Emacs for inclusion in
> a release (otherwise ELPA complains). I think that would be 29.4.
>
> That's the convenience of having it in a branch though; we can get to it
> at any point.
>
> > On Mon, 13 Jan 2025 at 23:07, Fran Burstall (Gmail) <
> [email protected]>
> > wrote:
> >
> >> I get a compilation error:
> >>
> >>
> >>     "emms-radio-browser.el:336:2: Error: Need command, got ‘:info’"
> >>
> >>
> >>  Hmmm,  I cannot reproduce.  Starting from emacs -Q, I add the emms dir
> to
> >> the loadpath and then bytecompile the emms-radio-browser file.  I get no
> >> errors.  This is on emacs 30.0.93.
> >>
> >> ---Fran
> >>
> >>
> >>
> >> On Mon, 13 Jan 2025 at 21:19, Yoni Rabkin <[email protected]> wrote:
> >>
> >>> "Fran Burstall (Gmail)" <[email protected]> writes:
> >>>
> >>> >>
> >>> >> Can you please open a branch with this code in it so that we can
> test
> >>> >> and try it out easily?
> >>> >
> >>> >
> >>> > Yup: the 'radio-browser' branch should be ready for testing now.
> >>>
> >>> The feature looks great, but I get a compilation error:
> >>>
> >>>     "emms-radio-browser.el:336:2: Error: Need command, got ‘:info’"
> >>>
> >>>
> >>> > On Sat, 11 Jan 2025 at 19:33, Yoni Rabkin <[email protected]> wrote:
> >>> >
> >>> >> "Fran Burstall (Gmail)" <[email protected]> writes:
> >>> >>
> >>> >> > I started to play with the https://www.radio-browser.info API and
> >>> built
> >>> >> a
> >>> >> > radio station browser for EMMS which I attach.
> >>> >> >
> >>> >> > There are three entry points
> >>> >> >
> >>> >> > emms-radio-browser-search-by-name
> >>> >> > emms-radio-browser-search-by-url
> >>> >> > emms-radio-browser-full-search
> >>> >> >
> >>> >> > All search the radio-browser database and return a playlist of
> >>> results.
> >>> >> > The last of these needs the (built-in since v28.1) transient
> package.
> >>> >> >
> >>> >> > It has only been lightly tested.
> >>> >> >
> >>> >> > If you think this is a worthwhile addition to EMMS, I can add it
> to
> >>> the
> >>> >> git
> >>> >> > repo and write some documentation...
> >>> >> >
> >>> >> > ---Fran
> >>> >>
> >>> >> I think that sounds good; thank you for working on this.
> >>> >>
> >>> >> Can you please open a branch with this code in it so that we can
> test
> >>> >> and try it out easily?
> >>> >>
> >>> >>
> >>> >> > On Thu, 26 Dec 2024 at 17:11, Fran Burstall (Gmail) <
> >>> >> [email protected]>
> >>> >> > wrote:
> >>> >> >
> >>> >> >> I have been playing with emms-streams and with
> >>> >> >>
> >>> >> >>    (setopt emms-player-mpv-update-metadata t)
> >>> >> >>
> >>> >> >> it is very capable.
> >>> >> >>
> >>> >> >> One mild pain point however is that adding a new stream with
> >>> >> >> 'emms-add-streamlist' and friends gives a streamlist with less
> >>> >> information
> >>> >> >> than the built-in streamlists: it lacks the metadata field which
> is
> >>> >> useful
> >>> >> >> for getting the station name (which can then be fed to a
> >>> >> track-description
> >>> >> >> function to make for a more informative display in the playlist
> >>> buffer).
> >>> >> >>
> >>> >> >> Of course, one can populate such a field by hand and this is
> what I
> >>> have
> >>> >> >> been doing but There Must Be A Better Way.  This is the sort of
> >>> thing
> >>> >> that
> >>> >> >> the unimplemented emms-streams-info.el could be doing but I
> >>> understand
> >>> >> that
> >>> >> >> querying the url for such information is a bit of a nightmare
> (does
> >>> the
> >>> >> >> stream have ICY tags etc).  There have been previous discussions
> on
> >>> this
> >>> >> >> list about this.
> >>> >> >>
> >>> >> >> However, there seems to be an alternative:
> >>> >> https://www.radio-browser.info
> >>> >> >> is a free (as in freedom, as far as I can tell) repository of
> >>> station
> >>> >> >> information with an API.  Perhaps one could query this to get
> >>> metadata
> >>> >> >> about the stream?
> >>> >> >>
> >>> >> >> One could also imagine other uses of this data like a radio
> station
> >>> >> >> browser in EMMS or being able to add streams by name rather than
> >>> url...
> >>> >> >>
> >>> >> >> Thoughts?  Worth pursuing?
> >>> >> >>
> >>> >> >> ---Fran
> >>> >> >>
> >>> >> >>
> >>> >> >>
> >>> >> >
> >>> >>
> >>> >> --
> >>> >>    "Cut your own wood and it will warm you twice"
> >>> >>
> >>>
> >>> --
> >>>    "Cut your own wood and it will warm you twice"
> >>>
> >>
>
> --
>    "Cut your own wood and it will warm you twice"
>

Reply via email to