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

[win32][depends]Patch TagLib to v1.11.1 + d2e0e55 to fix regression causing hangs #11522

Merged
merged 1 commit into from
Jan 27, 2017

Conversation

DaveTBlake
Copy link
Member

@DaveTBlake DaveTBlake commented Jan 22, 2017

For win32 update TagLib to v1.11.1 with taglib/taglib@d2e0e55 as a patch to fix regression causing hangs with (Shoutcast) internet radio streams. See ticket http://trac.kodi.tv/ticket/17210 .
(Shoutcast) internet radio streams ending ".mp3" get passed to TabLib as if they have ID3 tags, and TabLib was spending upto 10 mins looking for them, looking for the end of the file incase the tags were there.

The step to v1.11.1 brings win32 in line with the package changes for other platforms made by #11101
and the patch to match #11543

This change has already been backported to v17 by #11547

Note for future update to TagLib - it is worth testing with Shoutcasts, as well as other music file formats.

@DaveTBlake
Copy link
Member Author

Thanks @Paxxi for guiding me through this.

@DaveTBlake DaveTBlake changed the title [win32][depends]Update taglib to v1.11.1 [win32][depends]Patch TagLib to v1.11.1 + d2e0e55 to fix regression causing hangs Jan 27, 2017
@DaveTBlake
Copy link
Member Author

Caught master up with Krypton changes, thanks @Paxxi and @Rechi.
Makes sense to merge this @MartijnKaijser

@Paxxi Paxxi merged commit fc92c40 into xbmc:master Jan 27, 2017
@DaveTBlake DaveTBlake deleted the taglib1_11_1 branch January 27, 2017 13:57
@MartijnKaijser MartijnKaijser modified the milestone: L 18.0-alpha1 Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants