-
-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
error when using delay with single #4265
Comments
???? |
How would you do that? @vitoyucepi |
This is happening for me too (v2.3.0 Docker). Seems any source when paired with |
Here's my minimal repro:
|
Thanks y'all I'm gonna have a look shortly. |
Seems this isn't limited to |
This problem was introduced by 77cf76e, similar to #4262, but not fixed in #4270. Bisecting made me feel miserable.--- a/dune-project
+++ b/dune-project
@@ -45,18 +45,18 @@
(name liquidsoap-core)
(depends
(ocaml (>= 4.14.0))
- (dtools (>= 0.4.5))
- (duppy (>= 0.9.3))
- (mm (>= 0.8.4))
- (re (>= 1.11.0))
- (ocurl (>= 0.9.2))
- (cry (>= 1.0.0))
- (camomile (>= 2.0.0))
+ (dtools (= 0.4.5))
+ (duppy (= 0.9.3))
+ (mm (= 0.8.4))
+ (re (= 1.11.0))
+ (ocurl (= 0.9.2))
+ (cry (= 1.0.0))
+ (camomile (= 2.0.0))
uri
fileutils
menhirLib
- (saturn_lockfree (>= 0.4.1))
- (metadata (>= 0.2.0))
+ (saturn_lockfree (= 0.4.1))
+ (metadata (= 0.2.0))
dune-build-info
(liquidsoap-lang (= :version))
(ppx_string :build)) |
Happy to report that this is fixed by: #4289 thanks for the patience y'all. |
Hi @toots, test = single("music/1.mp3")
radio = mksafe(delay(20., test))
output.dummy(radio)
thread.run({print(metadata.cover.remove(radio.last_metadata() ?? []))}, every=1.)
settings.request.deprecated_on_air_metadata := true ffmpeg -f lavfi -i "sine=frequency=200:duration=10" -b:a 320k -c:a mp3 1.mp3 Should be 20 seconds of blank, 10 seconds of 1.mp3. |
test = single("music/1.mp3")
radio = mksafe(delay(10., test))
output.dummy(radio)
thread.run({print("#{time.up()} #{metadata.cover.remove(radio.last_metadata() ?? [])}")}, every=1.)
settings.init.allow_root := true
%ifversion >= 2.3
settings.request.deprecated_on_air_metadata := true
%endif
|
The script behavior appears correct to me:
You can still revert to the old behavior using the Thus, this script appears correct w.r.t to the two points above:
I added a new migration note for this in e262927 |
The better test would be:
Similar to how I did it above. This wasn't working in 2.3. Once this fix hits the rolling release I can test it. |
Description
In version 2.2.5, the delay was used together with the single and with that the metadata was updated normally, however, in version 2.3.0, the metadata is no longer updated.
Steps to reproduce
Expected behavior
I would like to know how to display it again
Liquidsoap version
Liquidsoap build config
Installation method
From OPAM
Additional Info
No response
The text was updated successfully, but these errors were encountered: