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

[Diego Biurrun]

On Sun, Sep 09, 2007 at 10:48:38AM -0700, Dave Yeo wrote:
> Diego Biurrun wrote:
> ...
>> Patch in its current form is rejected.
> Lets try a different approach. Attached is the patch that I've been using.
> Pros
> As long as the SDL dlls are fairly new we no longer need the MorphToPM 
> stuff. Also need fairly new SDL and fslib for fullscreen to work.

Then we should just require new SDL stuff IMO.

> Cons
> make test creates files in $TMP then can fail as it egreps /tmp

That is a separate issue, patches welcome.

> --- ffmpeg/configure	Sun Sep  9 00:01:48 2007
> +++ ffmpeg.os2/configure	Sun Sep  9 08:55:20 2007
> @@ -1196,6 +1199,26 @@
>      targetos=irix
>      ranlib="echo ignoring ranlib"
>      ;;
> +  os/2)
> +    FFSERVERLDFLAGS=""

ffserver works?

> +    LIBPREF=""
> +    LIBSUF="_s.lib"
> +    SLIBPREF=""
> +    SLIBSUF=".dll"
> +    SLIBNAME_WITH_MAJOR='$(SLIBPREF)$(NAME)-$(LIBMAJOR)$(SLIBSUF)'
> +    SLIB_EXTRA_CMD="dllar -o \$(NAME)\$(SLIBSUF) \$(NAME)\$(LIBSUF) -d \"\$(SLIBNAME_WITH_MAJOR)\" -omf -ord \$(LDFLAGS) \$(EXTRALIBS) \$(EXTRAOBJS); cp -f \$(SLIBNAME) \$(SLIBNAME_WITH_MAJOR)"
> +    SLIB_INSTALL_EXTRA_CMD="-install -m 644 \$(SLIBNAME:\$(SLIBSUF)=.lib) \$(SLIBNAME:\$(SLIBSUF)=.a)"

Maybe you can explain the tricks you are playing here...

Apart from that the patch looks OK to me.  Mans?

Diego