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

Build fails with older libmpdclient #14

Open
spendist opened this issue Mar 7, 2024 · 1 comment
Open

Build fails with older libmpdclient #14

spendist opened this issue Mar 7, 2024 · 1 comment

Comments

@spendist
Copy link

spendist commented Mar 7, 2024

In file included from mpd_control.cpp:13:
mpd_control.hpp:19:61: note: '#pragma message: Compiling with eventfd polling support.'
19 | #pragma message ( "Compiling with eventfd polling support." )
| ^
mpd_control.cpp: In member function 'std::optional<dynamic_image_data> mpd_control::get_albumart(std::string)':
mpd_control.cpp:424:29: error: 'mpd_send_albumart' was not declared in this scope; did you mean 'mpd_send_clear'?
424 | return get_cover(path, &mpd_send_albumart, &mpd_recv_albumart, &mpd_run_albumart);
| ^~~~~~~~~~~~~~~~~
| mpd_send_clear
mpd_control.cpp:424:49: error: 'mpd_recv_albumart' was not declared in this scope; did you mean 'mpd_recv_binary'?
424 | return get_cover(path, &mpd_send_albumart, &mpd_recv_albumart, &mpd_run_albumart);
| ^~~~~~~~~~~~~~~~~
| mpd_recv_binary
mpd_control.cpp:424:69: error: 'mpd_run_albumart' was not declared in this scope; did you mean 'mpd_run_clear'?
424 | return get_cover(path, &mpd_send_albumart, &mpd_recv_albumart, &mpd_run_albumart);
| ^~~~~~~~~~~~~~~~
| mpd_run_clear
mpd_control.cpp: In member function 'std::optional<dynamic_image_data> mpd_control::get_readpicture(std::string)':
mpd_control.cpp:429:29: error: 'mpd_send_readpicture' was not declared in this scope; did you mean 'get_readpicture'?
429 | return get_cover(path, &mpd_send_readpicture, &mpd_recv_readpicture, &mpd_run_readpicture);
| ^~~~~~~~~~~~~~~~~~~~
| get_readpicture
mpd_control.cpp:429:52: error: 'mpd_recv_readpicture' was not declared in this scope; did you mean 'get_readpicture'?
429 | return get_cover(path, &mpd_send_readpicture, &mpd_recv_readpicture, &mpd_run_readpicture);
| ^~~~~~~~~~~~~~~~~~~~
| get_readpicture
mpd_control.cpp:429:75: error: 'mpd_run_readpicture' was not declared in this scope; did you mean 'get_readpicture'?
429 | return get_cover(path, &mpd_send_readpicture, &mpd_recv_readpicture, &mpd_run_readpicture);
| ^~~~~~~~~~~~~~~~~~~
| get_readpicture
make: *** [Makefile:663: mpd_touch_screen_gui-mpd_control.o] Error 1

@muesli4
Copy link
Owner

muesli4 commented Mar 7, 2024

Hi @spendist! Your version of libmpdclient is probably too old. This is a newer MPD command to fetch the albumart directly from MPD. If you do not need this feature you can checkout d654d13, then it should compile.

  • It may be possible to introduce a configure option for this. But I am not sure if it is worth the hassle.
  • Alternatively, version requirements will be added.

See https://github.com/MusicPlayerDaemon/libmpdclient/blob/30dd8d029ccda0dce07e6b57ed7ec8b23bd59a48/NEWS#L23.

@muesli4 muesli4 changed the title Make error on Raspberry pi , Build fails with older libmpdclient Mar 8, 2024
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

2 participants