[M3devel] [M3commit] CVS Update: cm3

[Randy Coleburn]

Jay:
 
I'm not sure why an m3overrides file is needed for "some builds".
 
The WindowsResources package simply adds a template that allows folks on Windows platforms to include icons and other resources in their programs.  This package shouldn't be needed for non-Windows platforms.
 
Indeed, the template (winRes.tmpl) uses "if equal (OS_TYPE, "WIN32")" to bracket all of its activity, so it shouldn't do anything on non-Windows platforms.
 
Perhaps, the same logic should be used in the m3makefile to prevent inclusion of the WindowsResource package on any platform other than Windows, but then that would probably force similar logic in all packages that import WindowsResources.  I know that CM3IDE uses WindowsResources and, of course, I have a number of programs that use it.
 
Regards,
Randy

>>> Jay Krell <jkrell at elego.de> 12/31/2008 11:57 AM >>>
CVSROOT:/usr/cvs
Changes by:jkrell at birch.08/12/31 11:57:14

Added files:
cm3/m3-sys/windowsResources/src/: m3overrides 

Log message:
Add missing override to fix some builds.

I'm highly suspicious of all this "override" stuff.
It clearly causes too much duplication and distribution of information.
I shouldn't have to know the directory structure of my dependencies,
and even if I do, I should be able to share that knowledge with their many
other dependents. The "scripts" directory also figures this sort of stuff out automatically..
Being able to have multiple package stores is well and good.
I'm not sure they need to ever be used in an "unshipped" form, but instead just
use alternate roots and create new empty roots as needed. ?


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://m3lists.elegosoft.com/pipermail/m3devel/attachments/20081231/d2ca6346/attachment-0002.html>