search

jcorporation / myMPD

myMPD is a standalone and mobile friendly web mpd client with a tiny footprint and advanced features.
https://jcorporation.github.io/myMPD/
GNU General Public License v3.0
418 stars 65 forks source link

No coverimage for devel build #119

Closed parkmino closed 5 years ago

parkmino commented 5 years ago

No coverimage and sometimes with the following error message. ERROR Error extracting coverimage, invalid music_directory

I have no idea why the local downloaded src files from github are engaged here, is it my fault?

Ubuntu 18.04.2 X86_64

mympd.conf.txt

log.txt

jcorporation commented 5 years ago

You use embedded coverimages? I don't see the error in your provided logs.

In debug mode myMPD prints the source code file and line in the log, that is intended.

parkmino commented 5 years ago

Yeah, embedded covers does not show in devel while external ones are fine.

parkmino commented 5 years ago

There might have been some changes and now I see most of all cover, however not all. I will look into it more.

jcorporation commented 5 years ago

Devel code is in flow ;) today i am worked on making the music directory configurable in the gui.

parkmino commented 5 years ago

Is there any way to resolve the following error? I cannot change the mpd connection in gui.

ERROR    MPD connection: incorrect password
ERROR    MPD connection failed
parkmino commented 5 years ago

I removed /var/lib/mympd/state/mpd_pass and it works again. I will look into it.

parkmino commented 5 years ago

I am confused... all covers are not shown and just showing 'not playing'

parkmino commented 5 years ago

New error There's no sticker setting in mympd.conf

ERROR    MPD error: unknown command "sticker"
ERROR    MPD error mpd_send_sticker_set: unknown command "sticker"
ERROR    MPD error mpd_run_sticker_set: unknown command "sticker"
parkmino commented 5 years ago

Is it related to the latest commit? https://github.com/jcorporation/myMPD/commit/4d2bacc652a7178e15940a7406ee9d51694ae34d

jcorporation commented 5 years ago

No, but the sticker feature detection was broken. I fixed it with commit 0d3021d3d6eb4986ffcac62cf4c2a1ee2742df26 Stickers can be also disabled in settings in the gui

parkmino commented 5 years ago

With the latest devel, most of embedded cover shows except the attached one. Could you please check this on your side?

이진아 - Always with us (Inspired by 파랑).mp3.zip

parkmino commented 5 years ago

The album cover exists in /var/lib/mympd/covercache/, however no show.

jcorporation commented 5 years ago

I tested it with recent devel and I see the cover. Can you attach the debug logs?

parkmino commented 5 years ago

It is stange that I see the cover at this time, close this issue then.