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

make fails with libav 11.3 #128

Closed
freultwah opened this issue Apr 3, 2015 · 2 comments
Closed

make fails with libav 11.3 #128

freultwah opened this issue Apr 3, 2015 · 2 comments

Comments

@freultwah
Copy link
Contributor

Trying to build forked-daapd against libav 11.3 results in the following:

uffer.o test -f 'ffmpeg_url_evbuffer.c' || echo './'ffmpeg_url_evbuffer.c
ffmpeg_url_evbuffer.c:40:19: error: unknown type name 'URLContext'
url_evbuffer_open(URLContext *h, const char *filename, int flags)
^
ffmpeg_url_evbuffer.c:46:16: error: use of undeclared identifier 'URL_WRONLY'
if (flags != URL_WRONLY)
^
ffmpeg_url_evbuffer.c:95:20: error: unknown type name 'URLContext'
url_evbuffer_close(URLContext *h)
^
ffmpeg_url_evbuffer.c:103:20: error: unknown type name 'URLContext'
url_evbuffer_write(URLContext *h, unsigned char *buf, int size)
^
ffmpeg_url_evbuffer.c:122:1: error: unknown type name 'URLProtocol'
URLProtocol evbuffer_protocol = {
^

It appears that URLContext and URLProtocol were made internal in ffmpeg and completely thrown out of libav:

sackmotion/motion#15

Neither struct is to be found anywhere in libav includes.

@ejurgensen
Copy link
Member

Yes, I think they have actually been internal for while. The problem here is that ffmpeg_url_evbuffer is even included in the build, because the check in configure.ac should make COND_AVIO=true, which means that avio_evbuffer should get built instead of ffmpeg_url_evbuffer. I don't know why that doesn't work in this case (maybe something with pkg_config?). A quick work-around is to just change src/Makefile.am, so that the right file is built.

I may make that change in master too, I think some of the ffmpeg versions forked-daapd is trying to support are pretty ancient by now.

@ejurgensen
Copy link
Member

I've removed support for these old versions of ffmpeg now.

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

No branches or pull requests

2 participants