Skip to content
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

Builtin metadata.map is not registered when parsing config file #3423

Closed
rmens opened this issue Sep 23, 2023 · 2 comments
Closed

Builtin metadata.map is not registered when parsing config file #3423

rmens opened this issue Sep 23, 2023 · 2 comments

Comments

@rmens
Copy link

rmens commented Sep 23, 2023

Describe the bug
Parsing the config file generates an error in Liquidsoap 2.2.1.

To Reproduce
Create an empty .liq file and try to parse that:

root@liq:/var/log/liquidsoap# touch /tmp/empty.liq
root@liq:/var/log/liquidsoap# liquidsoap -p /tmp/empty.liq 
INFO: Loading Sdl_image, Target = linux
INFO: Loading Sdl_ttf, Target = linux
At /usr/share/liquidsoap-lang/libs/extra/deprecations.liq, line 96, char 17-38:
def map_metadata(%argsof(metadata.map), fn, s)

Error 3: Builtin metadata.map is not registered!

Expected behavior
Parse the file without errors.

Version details

  • OS: Debian
  • Version 22.04 (Liquidsoap 2.2.1)

Install method
DEB package

@vitoyucepi
Copy link
Collaborator

vitoyucepi commented Sep 23, 2023

I can confirm this problem.
Liquidsoap in parse-only mode prints an error.

Versions: 88380cf, 2.2.1, 2.1.4, 2.0.7.

To reproduce
echo '' | liquidsoap --parse-only -

@toots
Copy link
Member

toots commented Sep 26, 2023

Thanks for reporting. I have fixed the issue by disabling the deprecated layer when running in parse-only mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants