Debian build fixes #4358

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

rbalint commented Mar 7, 2014

#4354 against master.

Owner

MartijnKaijser commented Mar 7, 2014

did you actually read what i said in the other PR?

Contributor

rbalint commented Mar 7, 2014

Have you noticed that I have even answered to it?;-)

Owner

MartijnKaijser commented Mar 7, 2014

Yes and you still don't seem to get the idea: No external FFMPEG until we feel comfortable to provide it.
Do whatever you want with debian and use this patch for yourself.

Contributor

rbalint commented Mar 7, 2014

Hmm, what is wrong with the Debian packages?
You don't have to support them and it is used by many happy users?

Owner

MartijnKaijser commented Mar 7, 2014

then we don't need this code either as you provide packages for Debian we do not support.
besides after gotham is released the inclusion of ffmpeg will change and this becomes obsolete anyway so why merge it.

Contributor

rbalint commented Mar 7, 2014

Have you noticed that not every commit is about ffmpeg?

Owner

MartijnKaijser commented Mar 7, 2014

then split those out in a separate PR

Contributor

rbalint commented Mar 7, 2014

So what is the plan with ffmpeg? Using its intarnal API instead of encouraging the maintenance of an API which would fit other users of ffmpeg libs?

Member

FernetMenta commented Mar 7, 2014

FernetMenta closed this Mar 7, 2014

Contributor

rbalint commented Mar 7, 2014

If and when FFmpeg becomes available in Debian testing I can switch the XBMC packages to using it. But to allow that being able to use ffmpeg without too much patching would be handy.
The direction in FernetMenta's branch also helps, at least XBMC uses vanilla FFmpeg.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment