| Re: [AD] msvc makefile patch |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On Saturday 28 January 2006 09:57, Matthew Leverton wrote:
> The attached patches update the makefile process in the following ways:
hip hip hurray!
> *** Some points that people may want to discuss before applying:
>
> A) Naming convention of the static C runtime builds.
>
> These are only built if the user defines STATICRUNTIME. Two
> suggestions are: $(VERY_SHORT_NAME)srt and $(SHORT_VERSION)sr. The
> former is what the patch uses. For example, it would create:
> alsrt42.dll, alsrt.lib, alstr_s.lib, ... For most practical purposes,
> only the static release version would ever be used. (alsrt_s.lib).
I think it's ok, though it looks a bit ugly... what would the
$(SHORT_VERSION)sr look like? alleg42sr.dll &c?
That should be fine too...
> B) Whether INIT_CMD should exist, and if so, where should it really
> go... With the inclusion of the new names for the static runtime
> versions of MSVC, six new object folders need to be created. That's a
> lot of junk that hardly anyone will use. So instead of creating dummy
> folders, I set INIT_CMD to "mkdir" them on demand. It's executed under
> the "msg" target since that appears to always be the first thing done.
I think I would prefer to add a `dirs' target that creates the directories
if they do not exist and add an explicit dependency on that.
Evert