search

margro / xbmc

XBMC PVR Repository for the development of the MediaPortal and ForTheRecord pvr addons
http://www.scintilla.utwente.nl/~marcelg/xbmc/
Other
32 stars 14 forks source link

Terughoudend.... #10

Closed Red-F closed 13 years ago

Red-F commented 13 years ago

Hoi Marcel,

ik heb aardig wat aangepast en had halverwege juli een aardige versie die ook op OSX en Linux draaide met RTSP. Helaas hebben de wijzigingen tijdens mijn vakantie die versie hopeloos om zeep geholpen. Het compileert prima maar werkt totaal niet meer :)

Ik vermoed dat ik ook maar de LIVE555 lib ga gebruiken om de afhankelijkheden van XBMC wat te verminderen. Kost me wel tijd die ik eigenlijk op dit moment niet heb. Maar dat is een algemeen probleem.

Omdat het me wel handig leek als onze repo's weer wat meer in sync lopen bij deze dus een pull request.

Groeten, Fred

margro commented 13 years ago

Fred, could you please rebase these changes against my current master. I don't want to merge those "Merge remote branch" merge commit.

By the way, I was aware that you were working on RTSP support so although I've done some changes in the TSReader for MediaPortal purposes, I did not expect that it would break the ForTheRecord addon at your side.

margro commented 13 years ago

Just a quick note for the crossplatform future of the Mediaportal addons which may be of interest for ForTheRecord as well: My plan is also to make the TSReader code step-by-step cross platform and my first goal there is to start with getting the LIVE555 library compiling and running under Linux/MacOSX. I will test this first for MediaPortal. When this is done, I'm going to merge the two addons into 1 version and just use a configuration option to select ffmpeg-rtsp or LIVE555/TSReader.

Red-F commented 13 years ago

Ok, I rebased locally with upstream/master and then push --force 'd with my remote repo. That removed a lot of my history and probably did what you wanted....

margro commented 13 years ago

I've merged your changes and added a compile fix for the release build. Cheers!