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

Compatibility with TagLib 2.0 #7314

Closed
wants to merge 1 commit into from

Conversation

bilde2910
Copy link
Contributor

TagLib recently updated to version 2.0 and introduced breaking changes due to function deprecations and type changes. This patch updates Clementine to be compatible with TagLib 2.0.

I can confirm that this builds and works on my Arch Linux machine with TagLib 2.0, but I haven't tested it with TagLib 1.x. More testing may be warranted if support for TagLib 1.x is desired.

Fixes #7313.

@hatstand
Copy link
Contributor

More testing may be warranted if support for TagLib 1.x is desired.

I think the checks above give you your answer there :-)

@Pro-pra
Copy link

Pro-pra commented Mar 31, 2024

Thank you, may build fine with this patch

@PPN-SD
Copy link
Contributor

PPN-SD commented Dec 23, 2024

Hi,
I used the patch from @bilde2910 and added check version for taglib-1.
Tested with taglib-1.13.1 for gentoo package.
https://github.com/gentoo/gentoo/blob/29c3eb83f82bc78362f29b75221cf2e7b8e15c42/media-sound/clementine/files/clementine-1.4.0_rc2-fix-build-taglib2.patch

DarthGandalf pushed a commit to DarthGandalf/Clementine that referenced this pull request Jan 9, 2025
@hatstand hatstand closed this in 658f34e Jan 9, 2025
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

Successfully merging this pull request may close these issues.

Build fails with taglib 2.0
4 participants