[FFmpeg-devel] [PATCH] Re-add OS/2 support
Mon Sep 10 12:15:17 CEST 2007
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.
> 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.
> 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 @@
> ranlib="echo ignoring ranlib"
> + os/2)
> + FFSERVERLDFLAGS=""
> + 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?
More information about the ffmpeg-devel