Liquidsoap

reload playlist on change

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Liquidsoap
  • Labels:
    None

Description

It would be nice to be able to reload a playlist as soon as it changes (instead of periodically reloading it). I think that there are functions to monitor file changes.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Andrey Podshivalov added a comment - - edited
Concerning implementation, there's [http://tab.snarc.org/projects/ocaml_inotify/ a binding for inotify] and a command inotifywait in the inotify-tools package.

Good idea. Concerning the implementation, I'd be tempted to simplify playlist() and provide in the language what's needed to obtain desired features. I think the reload option in playlist can be simplified to only deal with the "round" mode, because this one can't be done externally. For time-based reloading, one can use add_timeout(): the result is slightly different but only better (without add_timeout, the playlist only reloads when it's played). Finally we could add reloading upon file change by binding inotify to the script language, I'm sure it'll find other applications too.
Show
Andrey Podshivalov added a comment - - edited Concerning implementation, there's [ http://tab.snarc.org/projects/ocaml_inotify/ a binding for inotify] and a command inotifywait in the inotify-tools package. Good idea. Concerning the implementation, I'd be tempted to simplify playlist() and provide in the language what's needed to obtain desired features. I think the reload option in playlist can be simplified to only deal with the "round" mode, because this one can't be done externally. For time-based reloading, one can use add_timeout(): the result is slightly different but only better (without add_timeout, the playlist only reloads when it's played). Finally we could add reloading upon file change by binding inotify to the script language, I'm sure it'll find other applications too.
Hide
Permalink
Samuel Mimram added a comment -
Implemented in "inotify" branch.
Show
Samuel Mimram added a comment - Implemented in "inotify" branch.
Hide
Permalink
Samuel Mimram added a comment -
Fixed in 52ddcab5ef45d177135a62d80e9718b36d7a27f3.
Show
Samuel Mimram added a comment - Fixed in 52ddcab5ef45d177135a62d80e9718b36d7a27f3.

People

  • Assignee:
    Samuel Mimram
    Reporter:
    Samuel Mimram
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: