> > +enabled coreimage_filter && { check_header_objcc QuartzCore/CoreImage.h > || disable coreimage_filter; } > +enabled coreimagesrc_filter && { check_header_objcc > QuartzCore/CoreImage.h || disable coreimagesrc_filter; } >
Wouldnt it be simpler to just add an item to HEADERS_LIST for QuartzCore_CoreImage_h then this check only needs to be done once as both filters can then just depend on the header list entry. e.g. coreimage_filter_deps="QuartzCore_CoreImage_h" ... check_header_objcc QuartzCore/CoreImage.h This would also remove the need for the duplicate _extralibs= lines aswell. VideoToolbox currently does something like this although much more complicated as it was added as an external lib despite being in a similar boat to coreimage. FYI I noticed what looks like an error in check_header_objcc as the code "enable_safe $headers" I believe should be "enable_safe $header" (no 's'). I dont have OSX to double check this. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel