[FFmpeg-devel] [PATCH] Proper support for building MSVC import libraries

Ramiro Ribeiro Polla ramiro
Sat Jul 21 00:10:05 CEST 2007


Diego Biurrun wrote:
> On Thu, Jul 19, 2007 at 06:36:14PM -0300, Ramiro Polla wrote:
>   
>> IMO, Makefile commands don't belong in configure (such as 
>> SLIB_INSTALL_EXTRA_CMD).
>>     
>
> I disagree.  I moved it to configure so that all system-specific
> settings are decided and set inside configure without affecting the
> Makefiles.
>
>   

Then how about we move install-lib-shared from common.mak into a 
one-line variable in configure too? All that symlinking and version 
number after the file name non-sense are system-specific for UNIX. It's 
wrong for MinGW.

(yes, I am being sarcastic)

I agree they should not be ifeq'd in the Makefiles like they used to be. 
That was ugly. But a Makefile command should not be entirely in 
configure. It is also ugly. Look at all the backslashes for special 
symbols. And the - sign before the command. Why does configure have to 
care about Makefile syntax?

Now look at BUILD_SHARED and BUILD_STATIC. Why doesn't configure just 
export INSTLIBTARGETS with the appropriate targets? Because that's a 
proper way of separating configure stuff from Makefile stuff.

What I propose to clean all this is to put Makefile commands in the 
Makefiles. If it's unacceptably ugly for Unix users to look at any 
Windows specific stuff in a Makefile, then just separate it into 
something like common_win32.mak. It won't be code duplication, just 
separation. To make a proper import lib target. There are some more 
stuff wrong for MinGW ATM. I am trying to clean MinGW32 support. It is 
ugly as is. It's hard to maintain since all MinGW specific stuff (hacks) 
are enclosed in one section in configure.

I know this will face fierce opposition, as it always does, but I am 
trying to clean it up, so, bear with me here...

Ramiro Polla




More information about the ffmpeg-devel mailing list