[FFmpeg-devel] [PATCH] define _BSD_SOURCE for bktr.c

Jacob Meuser jakemsr
Sat Dec 13 02:10:38 CET 2008


On Fri, Dec 12, 2008 at 11:52:24PM +0100, Reimar D?ffinger wrote:
> Hello,
> the dev/ic/bt8xx.h header needs types like u_short which for OpenBSD
> are defined by sys/types.h but only when _BSD_SOURCE is defined before it gets
> included.

what version of openbsd?  I maintain the openbsd port/package.  I have
never had an issue with this.  granted, I haven't tried updating
ffmpeg in a few months ...


> I haven't found a less ugly way to fix it but this:
> 
> diff --git a/libavdevice/bktr.c b/libavdevice/bktr.c
> index 7b37f11..7d9abee 100644
> --- a/libavdevice/bktr.c
> +++ b/libavdevice/bktr.c
> @@ -24,6 +24,7 @@
>   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
>   * 02110-1301 USA
>   */
>  
> +#define _BSD_SOURCE
>  #include "libavformat/avformat.h"
>  #if defined (HAVE_DEV_BKTR_IOCTL_METEOR_H) && defined (HAVE_DEV_BKTR_IOCTL_BT848_H)
>  # include <dev/bktr/ioctl_meteor.h>
> 
> 
> But since this is a BSD-specific file after all it might be appropriate
> (though I this "magic" could be done in the Makefile, too, I just don't
> know how).
> 
> Greetings,
> Reimar D?ffinger
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel

-- 
jakemsr at sdf.lonestar.org
SDF Public Access UNIX System - http://sdf.lonestar.org




More information about the ffmpeg-devel mailing list