Frank,<br><br>While I'm not sure about what changes you intend to do here, I would emphatically disagree removing the current approach with the ability of setting the full path to the external configuration file from the nmake command line. My build system is higly depend on this setting and if we don't have any compelling reason I would like to avoid relocating this custom file (which is anyway generated dynamically) into the gdal source tree.<br>
<br><br>Best regards,<br><br>Tamas<br><br><br><br><div class="gmail_quote">2011/2/11 Frank Warmerdam <span dir="ltr"><<a href="mailto:warmerdam@pobox.com">warmerdam@pobox.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Folks,<br>
<br>
Some time ago Tamas added a mechanism to nmake.opt (the central include<br>
file for windows MSVC nmake based makefiles) so that you could provide<br>
an extra definition file on the commandline.<br>
<br>
eg.<br>
nmake -f <a href="http://makefile.vc" target="_blank">makefile.vc</a> EXT_NMAKE_OPT=mynmake.opt<br>
<br>
While this is useful, expecially in cases where it is desirable to<br>
have several build configurations out of one tree, I never used it. I<br>
always just adding "!INCLUDE nmake.osgeo4w" or "!INCLUDE nmake.frank"<br>
at the beginning of nmake.opt and put my definitions in that new file.<br>
<br>
With Kirk's recent introduction of a complicated nmake.opt file for the<br>
MrSID driver, I've learned that you can check for the existance of a<br>
file in an nmake !IF EXISTS directive, and I have incorporated use of this<br>
in nmake.opt to pull in "nmake.local" if it exists.<br>
<br>
So going forward, I'm hoping the default way of tailoring windows builds<br>
will be to create an nmake.local file overriding definitions from nmake.opt.<br>
<br>
Best regards,<br>
-- <br>
---------------------------------------+--------------------------------------<br>
I set the clouds in motion - turn up | Frank Warmerdam, <a href="mailto:warmerdam@pobox.com" target="_blank">warmerdam@pobox.com</a><br>
light and sound - activate the windows | <a href="http://pobox.com/%7Ewarmerdam" target="_blank">http://pobox.com/~warmerdam</a><br>
and watch the world go round - Rush | Geospatial Programmer for Rent<br>
<br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
</blockquote></div><br><div style="visibility: hidden; left: -5000px; position: absolute; z-index: 9999; padding: 0px; margin-left: 0px; margin-top: 0px; overflow: hidden; word-wrap: break-word; color: black; font-size: 10px; text-align: left; line-height: 130%;" id="avg_ls_inline_popup">
</div>