[FFmpeg-devel] [PATCH] Enable proper IPv6 functions on windows

[Måns Rullgård]

Martin Storsj? <martin at martin.st> writes:

> Hi Ramiro,
>
> On Tue, 12 Jan 2010, Ramiro Polla wrote:
>
>> On Tue, Jan 12, 2010 at 7:57 AM, Martin Storsj? <martin at martin.st> wrote:
>> > The attached patches enable usage of the proper IPv6 enabled
>> > getaddrinfo/getnameinfo on windows, by disabling backwards compatibility
>> > with anything prior to Windows XP. This was OK'd by Ronald earlier.
>> >
>> > Ramiro, is this ok with you, too?
>> 
>> I think we should leave a configure option for win9x. Attached patch
>> also gives a one year deadline in the documentation for anyone with
>> such a need to speak up, otherwise we'll just remove win9x support
>> like many more programs are doing (latest one I saw was cygwin...).
>
> Here's a slightly modified version of your patch, renaming the value of 
> the --target-os parameter to mingw32-win9x (as suggested by Peter) and 
> clarifying the docs that this is needed for win2000 as well.
>
> Any other comments on this, or can it be applied?
>
> // Martin
>
> commit f2928a1bb3bda320f354c41cceeef3ef14824cf8
> Author: Martin Storsjo <martin at martin.st>
> Date:   Fri Jan 15 10:40:50 2010 +0200
>
>     Modified version of Ramiro's win9x patch
>
> diff --git a/configure b/configure
> index 6321bb9..e475bcd 100755
> --- a/configure
> +++ b/configure
> @@ -883,6 +883,7 @@ CONFIG_LIST="
>      vaapi
>      vdpau
>      version3
> +    win9x
>      x11grab
>      zlib
>  "

Why?  Will it be used somewhere later?

> @@ -2023,6 +2024,11 @@ case $target_os in
>              { check_cflags -mdynamic-no-pic && add_asflags -mdynamic-no-pic; }
>          ;;
>      mingw32*)
> +        if test $target_os = "mingw32-win9x"; then
> +            enable win9x
> +        else
> +            add_cflags -D_WIN32_WINNT=0x0501
> +        fi
>          if test $target_os = "mingw32ce"; then
>              disable network
>          else
> diff --git a/doc/general.texi b/doc/general.texi
> index 8e22969..5ead9a7 100644
> --- a/doc/general.texi
> +++ b/doc/general.texi
> @@ -996,6 +996,15 @@ and for a build with shared libraries
>  ./configure --target-os=mingw32 --enable-memalign-hack --enable-shared --disable-static --extra-cflags=-mno-cygwin --extra-libs=-mno-cygwin
>  @end example
>  
> + at subsection Win9x/Win2000 support
> +
> +FFmpeg is built by default for Windows XP and above. If you need to run FFmpeg
> +on Windows 9x or Windows 2000, use @code{--target-os=mingw32-win9x}. It will
> +disable some features that don't exist prior to Windows XP. Also please get
> +in touch with the ffmpeg-devel mailinglist so that we know someone with this
> +exotic need actually exists, otherwise Windows 9x/2000 support will be
> +removed by 2011.
> +
>  @section BeOS
>  
>  BeOS support is broken in mysterious ways.

-- 
M?ns Rullg?rd
mans at mansr.com