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

[Diego Biurrun]

On Sat, Oct 13, 2007 at 10:37:38PM -0700, Dave Yeo wrote:
> On 10/13/07 02:21 am, Diego Biurrun wrote:
>> On Tue, Oct 09, 2007 at 07:14:47PM -0700, Dave Yeo wrote:
>>> ...
> ...
>>> +    SLIB_UNINSTALL_EXTRA_CMD="rm -f 
>>> \$(SHLIBDIR)/\$(SLIBNAME_WITH_MAJOR:\$(SLIBSUF)=.lib)"
>>>      SHFLAGS="-shared 
>>> -Wl,--output-def,\$(@:${SLIBSUF}=.def),--out-implib,lib\$(SLIBNAME:\$(SLIBSUF)=.dll.a) 
>>> -Wl,--enable-runtime-pseudo-reloc -Wl,--enable-auto-image-base"
>> This does not apply anymore.  Also, you can save some of the backslashes
>> by using single quotes.
>>> --- common.mak	(revision 10693)
>>> +++ common.mak	(working copy)
>>> @@ -97,6 +97,7 @@
>>>  	       "$(SHLIBDIR)/$(SLIBNAME)"            \
>>>  	       "$(SHLIBDIR)/$(SLIBNAME_WITH_VERSION)"
>>>  	-rm -f "$(LIBDIR)/$(LIB)"
>>> +	$(SLIB_UNINSTALL_EXTRA_CMD)
>> This belongs one line up along with the other shared lib uninstall
>> stuff.
>> Looks OK otherwise.
>
> Ok I've left the mingw part out though once the windows people decide how 
> to handle installing the import libs etc they will most likely want to use 
> it.

Patch applied, also the MinGW part along with some simplifications and
corrections.

Diego