Looks good.<br><br>Tamas<br><br><br><br><div class="gmail_quote">2011/2/12 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;">
<div class="im">On 11-02-11 06:56 PM, Tamas Szekeres wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Frank,<br>
<br>
While I'm not sure about what changes you intend to do here, I would<br>
emphatically disagree removing the current approach with the ability of setting<br>
the full path to the external configuration file from the nmake command line.<br>
My build system is higly depend on this setting and if we don't have any<br>
compelling reason I would like to avoid relocating this custom file (which is<br>
anyway generated dynamically) into the gdal source tree.<br>
</blockquote>
<br>
<br></div>
Tamas,<br>
<br>
I did not remove the support for EXT_NMAKE_OPT - the first stuff in<br>
nmake.opt now looks like:<br>
<br>
###############################################################################<br>
# For convenience, user may put custom settings in a local settings file<br>
# named nmake.local, or a name defined by the EXT_NMAKE_OPT option.<br>
<br>
!IF EXIST("$(GDAL_ROOT)\nmake.local")<br>
!INCLUDE $(GDAL_ROOT)\nmake.local<br>
!ENDIF<br>
<br>
# nmake -f <a href="http://makefile.vc" target="_blank">makefile.vc</a> EXT_NMAKE_OPT=mynmake.opt<br>
!IFDEF EXT_NMAKE_OPT<br>
!INCLUDE $(EXT_NMAKE_OPT)<br>
!ENDIF<br>
<br>
I believe this will serve both needs smoothly, albeit with a little<br>
extra complication in the nmake.opt.<div><div></div><div class="h5"><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>
</div></div></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>