dweymouth / supersonic

A lightweight and full-featured cross-platform desktop client for self-hosted music servers
GNU General Public License v3.0
939 stars 46 forks source link

Starting play of songs does not work #112

Closed Gomez closed 1 year ago

Gomez commented 1 year ago

I build current main on my Debian 11.6 desktop. App works and i can login and see my albums.

But playing songs does not work, nothing happens when I press play.

Log shows this:

2023/04/03 11:19:11   At: /home/gomez/go/pkg/mod/github.com/dweymouth/fyne/v2@v2.3.0-rc1.0.20230331041414-b548301c117c/internal/svg/svg.go:27
2023/04/03 11:19:11 Fyne error:  could not load SVG, falling back to static content:
2023/04/03 11:19:11   Cause: XML syntax error on line 3: invalid character entity & (no semicolon)
2023/04/03 11:19:11   At: /home/gomez/go/pkg/mod/github.com/dweymouth/fyne/v2@v2.3.0-rc1.0.20230331041414-b548301c117c/internal/svg/svg.go:27
2023/04/03 11:19:16 error fetching image: image: unknown format
2023/04/03 11:19:16 error fetching image: image: unknown format
2023/04/03 11:19:16 error fetching image: image: unknown format
Xlib:  extension "NV-GLX" missing on display ":1".

Is there a way to get more debugging?

dweymouth commented 1 year ago

I definitely do need to add more debugging log capabilities. Some more information might help to figure out what's going on here:

It could possibly be related to #91 . Since you're building from source you could add a log to the beginning of the AppendFile and PlayFile functions here:

log.Printf("Adding playback URL: %s", url)

You'll also need to add the "log" package to the import block at the top of the file.

Then, build and run and try copying one of those URLs manually from the terminal and seeing if mpv can play it directly: mpv <url>

If MPV fails with an "unknown format" error or similar, it is #91 which I've only seen on airsonic(-advanced), and reproduces on other mpv-based Subsonic clients that I tested with.

anarcat commented 1 year ago

I also can't play audio using libmpv.1.so from the 0.32.0-3 Debian package, using the binary package of the latest beta release here.

that said, however, if i build the latest git tree in Debian after installing libmpv-dev, audio does play. the latest debian package in Debian is actually libmpv2, so maybe that's the problem here?

anarcat commented 1 year ago

It could possibly be related to #91 . Since you're building from source you could add a log to the beginning of the AppendFile and PlayFile functions here:

log.Printf("Adding playback URL: %s", url)

You'll also need to add the "log" package to the import block at the top of the file.

fwiw i did that in #130.

anarcat commented 1 year ago

i wonder if the problem here might not have been fixed on git head, in #121, because when i tested this outside of the flatpak, it failed with the beta release but worked from the git HEAD...

Gomez commented 1 year ago

Sorry for the delay. I can confirm the problem is fixed in 0.1.0-beta and the flatpak. Thanks @anarcat! And thanks @dweymouth!

Issue can be closed?

anarcat commented 1 year ago

as far as i am concerned, yes! i can't close it but you can :)