[FFmpeg-devel] [PATCH] Add SLIB_UNINSTALL_EXTRA_CMD to build system.

[Ramiro Polla]

Dave Yeo wrote:
> On 10/08/07 06:28 am, Diego Biurrun wrote:
> > On Sun, Oct 07, 2007 at 10:16:26PM -0700, Dave Yeo wrote:
> >> Diego Biurrun wrote:
> >>> On Sat, Sep 15, 2007 at 09:38:39PM -0700, Dave Yeo wrote:
> >>>> Diego Biurrun wrote:
> ...
> >>>> Note also that we (and also Windows?) need an extra uninstall 
> command as
> >>>> well in a future patch.
> >>> Patches welcome.
> >> Yes, well one patch at a time :)
> >
> > Let's see the patch right away.  If it is needed on Windows I'll likely
> > apply it right away.
> >
> ...
> I've attached a quick untested patch for mingw, should uninstall the
> same lib installed by SLIB_INSTALL_EXTRA_CMD. Though I am somewhat 
> confused as I thought the SLIB_INSTALL_EXTRA_CMD was installing an 
> import lib yet it seems to be installing the DLL with a .lib suffix 
> instead of .dll.

This is an MSVC import lib.

> Also it looks like gcc is creating an import lib with the suffix of 
> .dll.a which is not installed.
> Kept to the same style as used in the rest of mingw section of configure.
> Also cygwin has similar behaviour, looks like it creates an import lib 
> but never installs it.
> Do both cygwin and mingw allow directly linking against DLLs and if so 
> what is the point of creating import libs and in mingw case installing 
> the DLL with a lib suffix?

Reading [1], it seems OK to remove them. I'll apply attached patch 
tomorrow unless someone has a good reason to object.

Ramiro Polla
[1] http://sourceware.org/binutils/docs-2.18/ld/WIN32.html
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dll.a.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071013/2ff0d80a/attachment.txt>