[FFmpeg-devel] [PATCH] Re-add OS/2 support

[Diego Biurrun]

On Sat, Sep 08, 2007 at 04:37:44PM +0900, KO Myung-Hun wrote:
> 
> This patch re-add OS/2 support.
> 
> --- configure	(revision 10425)
> +++ configure	(working copy)
> @@ -696,6 +718,7 @@
>      mkstemp
>      mlib
> +    os2

This is unused and unnecessary.

> @@ -1199,6 +1223,27 @@
>      targetos=irix
>      ranlib="echo ignoring ranlib"
>      ;;
> +  os/2*)   
> +    TMPE=$TMPE".exe"   

This should rather be made part of the TMPE definition.

> +    FFSERVERLDFLAGS=""     
> +    ffserver="no"  

nonsense

> +    pkg_requires=""    

nonsense

> +    dv1394="no"    

redundant

> +    os2="yes"  

see above

> @@ -1641,7 +1686,7 @@
>      temp_extralibs `"${SDL_CONFIG}" --libs`
> -    if check_lib SDL.h SDL_Init; then
> +    if check_lib2 SDL.h SDL_Init; then

This looks wrong.

Also, there is a ton of trailing whitespace.

Patch in its current form is rejected.

Diego