arkq / cmusfm

Last.fm standalone scrobbler for the cmus music player
GNU General Public License v3.0
234 stars 5 forks source link

Scrobbling Björk specifically doesn't work. #17

Closed sunnaryt closed 6 years ago

sunnaryt commented 6 years ago

I have a pretty extensive library including Japanese music that scrobbles fine. The only thing I have an issue with is Björk, in which case nothing happens at all. The same music works in iTunes.

arkq commented 6 years ago

Could you post somewhere the file which does not work?

sunnaryt commented 6 years ago

Unfortunately this would be illegal, is there something else I can do?

arkq commented 6 years ago

Unfortunately this would be illegal

yeap :/

OK, I know what you can do. In this repo there is a file cmusfm.debug. Use this file as a scrobbler (it dumps metadata exposed by the cmus) and post here generated /tmp/cmusfm.debug file.

sunnaryt commented 6 years ago

Sure, looking over the log it created it seems as though it doesn't see an artist.

cmusfm.debug.txt

Here's metadata on one of the files in the album through this website I found(the first one logged): https://www.get-metadata.com/result/1abf7898-534d-46e4-8a82-0a83919f0c9a I'm not sure how long this site keeps these results up (or maybe you don't want to use this site) so here is the results of the page put in plain text as well: get-metadata result copy.txt

Cmus as well detects the artist as normal.

arkq commented 6 years ago

Looking over the log it created it seems as though it doesn't see an artist.

Yes, you're right. However, there is nothing I can do about this. Lines stored in the cmusfm.debug file depict cmusfm invocation by cmus. Apparently, cmus does not expose artist for these two affected files. What concerns me, is the fact, that the file path does not contain artist name as well:

Music/(2001) Vespertine/Hidden Place.mp3 vs Music/Black Marble - It's Immaterial (2016)/01 Interdiction.mp3

Are you sure, that cmus detects artist tag correctly? Maybe it is not a standard "artist" tag. Check, in which tag artist is stored for other files. Anyway, this issue should be reported for cmus - it's not a cmusfm bug.

arkq commented 6 years ago

It seems, that in the next cmus release, album artist will be exposed to the status program: https://github.com/cmus/cmus/commit/4df336aa866a43c91a129ca92c0b277b338287c6

I will make required changes to the cmusfm. This change will be according to the issue #16, so if album artist will be present, it will be preferred.

arkq commented 6 years ago

Please check current master snapshot of cmusfm with cmus compiled from fresh master branch.