On 2014/12/23 06:47, Jan Brittenson <[email protected]> wrote:
> Cloned the git master (last commit 7f3fecbdf581faf7e98135d5cac535dee6faedf5) 
> and when building get the
> following error with libavutil 52.3.0 installed:
> 
> make  all-am
> make[1]: Entering directory `/home/bson/src/mpd'
> g++ -DHAVE_CONFIG_H -I.  -DNDEBUG -I./src -isystem /usr/include/glib-2.0 
> -isystem /usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include 
> -DSYSTEM_CONFIG_FILE_LOCATION='"/usr/local/etc/mpd.conf"'              
> -I/usr/include/opus     -I/usr/include/libbinio    -D_GNU_SOURCE -std=gnu++0x 
> -pthread -g -O2 -fvisibility=hidden -fno-threadsafe-statics 
> -fmerge-all-constants -fno-exceptions -fno-rtti -ffast-math -ftree-vectorize 
> -ffunction-sections -fdata-sections -Wall -Wextra -Wmissing-declarations 
> -Wshadow -Wpointer-arith -Wcast-qual -Wwrite-strings -Wsign-compare -MT 
> src/decoder/plugins/libdecoder_a-FfmpegDecoderPlugin.o -MD -MP -MF 
> src/decoder/plugins/.deps/libdecoder_a-FfmpegDecoderPlugin.Tpo -c -o 
> src/decoder/plugins/libdecoder_a-FfmpegDecoderPlugin.o `test -f 
> 'src/decoder/plugins/FfmpegDecoderPlugin.cxx' || echo 
> './'`src/decoder/plugins/FfmpegDecoderPlugin.cxx
> In file included from src/decoder/plugins/FfmpegDecoderPlugin.cxx:30:0:
> ./src/lib/ffmpeg/Buffer.hxx: In destructor 
> ???FfmpegBuffer::~FfmpegBuffer()???:
> ./src/lib/ffmpeg/Buffer.hxx:53:15: error: ???av_free??? was not declared in 
> this scope
>    av_free(data);
>                ^
> make[1]: *** [src/decoder/plugins/libdecoder_a-FfmpegDecoderPlugin.o] Error 1
> make[1]: Leaving directory `/home/bson/src/mpd'
> make: *** [all] Error 2
> 
> Looking at Buffer.hxx it doesn???t seem as if I got libavutil/mem.h included. 
>  The following change gets it
> building cleanly:

It gets included, as you see in your own patch:

>  #include <libavutil/mem.h>
>  #else
> +#include <libavutil/mem.h>

You better move this out of the #ifdef, so you can merge the two
#include lines.
_______________________________________________
mpd-devel mailing list
[email protected]
http://mailman.blarg.de/listinfo/mpd-devel

Reply via email to