Created attachment 162549 [details] Song with Lyrics tag that doesn't show lyrics in Elisa Git master everything on Plasma 6 with Elisa STEPS TO REPRODUCE 1. Play a song with a "Lyrics" ID3 tag 2. Open Now Playing view OBSERVED RESULT No lyrics detected. They don't show up in the Metadata editor window, either. EXPECTED RESULT Lyrics in all relevant places. ADDITIONAL INFORMATION Sample song that reproduces the issue is attached. Possibly related to Bug 476067; maybe caused by the same thing?
I can't reproduce with that file. FWIW ogg containers use Vorbis comments to store metadata. Can you try with an mp3 to see if it also happens with ID3 tags?
I tried with an MP3 file too; same issue. Are you also building Elisa on top of Plasma 6? What's your Qt version? Mine's 6.5.2 right now.
On Wednesday, 25 October 2023 21:21:37 BST you wrote: > https://bugs.kde.org/show_bug.cgi?id=476068 > > --- Comment #2 from Nate Graham <nate@kde.org> --- > I tried with an MP3 file too; same issue. > > Are you also building Elisa on top of Plasma 6? What's your Qt version? > Mine's 6.5.2 right now. I'm running it on Plasma 5 still but I can give it a test on Plasma 6 soon. Qt is 6.6 here. Could you run the taglibextractortest and taglibwritertest for KFileMetadata? I can see they have a test for the lyrics field here https://invent.kde.org/frameworks/kfilemetadata/-/blob/master/autotests/taglibextractortest.cpp?ref_type=heads#L204
Btw check you have taglib-devel or libtag-devel installed too. I recently had issues because I forgot to install that package so Elisa wasn't scanning files properly. I didn't realise it for a while because the database was already populated using the distro version of Elisa.
Ah, that's definitely worth following up. I'm not building TagLib from source right now. I'll give it a go and see if that helps.
Yep, that was it. I ran into one gotcha: by default taglib only builds a static library (.a) rather than a shared library (.so) which caused kfilemetadata to fail during compilation because it couldn't link a static library into a dynamic one: [ 93%] Linking CXX shared module ../../bin/kf6/kfilemetadata/writers/kfilemetadata_taglibwriter.so /usr/bin/ld: /home/nate/kde/usr6/lib64/libtag.a(mpegfile.cpp.o): relocation R_X86_64_32S against `.rodata' can not be used when making a shared object; recompile with -fPIC /usr/bin/ld: failed to set dynamic section sizes: bad value collect2: error: ld returned 1 exit status It turns out that taglib has an off-by-default CMake option to build shared libs, and after doing that I had to actually delete the buil static libs that got installed earlier. After I did that, kfilemetadata and Elisa linkes against the .so and all was well again. I've added these to my kdesrc-buildrc file: # For TagLib, because Fedora doesn't have a Qt6 build of it yet module utfcpp repository git@github.com:nemtrif/utfcpp.git end module # For KFileMetadata and Elisa, because Fedora doesn't have a Qt6 build of it yet module taglib repository git@github.com:taglib/taglib.git cmake-options -DBUILD_SHARED_LIBS=TRUE end module And ensured that I build those from source every day as well. Thanks so much for pointing me on the right path, Jack!