Kodi Community Forum
[PATCH] Fix for all RAR and ZIP subtitles getting loaded - Printable Version

+- Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Development (https://forum.kodi.tv/forumdisplay.php?fid=32)
+--- Forum: Kodi Application (https://forum.kodi.tv/forumdisplay.php?fid=93)
+--- Thread: [PATCH] Fix for all RAR and ZIP subtitles getting loaded (/showthread.php?tid=124218)



[PATCH] Fix for all RAR and ZIP subtitles getting loaded - devkid - 2012-02-29

Hey guys,

this is meant as a patch for this ticket:
http://trac.xbmc.org/ticket/12719

I added the patch to the ticket. Thats my first patch to XBMC sources so handle with care Wink Comment are welcome.

Seeing this patch still making it into Eden would be awesome because this bug is bothering me for quite a while...


- jmarshall - 2012-02-29

Cheers - mind doing a pull req for this so that it gets out to the devs who deal with this code quickly?


- devkid - 2012-03-01

I am quite a git noob. I tried to do it and I hope I have it right now.
I cloned the repository on github and commited my patch into my new repo. Then I created a pull request from my repo to the original xbmc repo. Is this correct?

Btw there is a "merge" that is part of my pull request. No idea why, I am sorry for that Confused

746 (PR)


- jmarshall - 2012-03-01

Have commented on the pull req as to how to avoid it in future (and fix it now) Smile


- DDDamian - 2012-03-01

Congrats devkid Smile


- devkid - 2012-03-02

Wow, I even thought it was a cool idea to merge the xbmc's master in again before I did that pull request. Well, it was not Cool

I had to google a bit to find out what that "rebase" stuff means, but I feel enlightened now Smile And I think I have it right now. Big thanks gentlemen!


- jmarshall - 2012-03-02

Yup, looks good.


- mad-max - 2012-03-02

Looks better now with the commit...
Had same problems with my PR a few days ago...
Got it sorted with the awesome help ( and patience ) from jmarshall...