<br><br><div class="gmail_quote">2011/1/13 Kirk McKelvey <span dir="ltr"><<a href="mailto:kmckelvey@lizardtech.com">kmckelvey@lizardtech.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 link="blue" vlink="purple" lang="FR"><div><div class="im"><p class="MsoNormal" style="margin-left: 35.4pt;"><span lang="EN-US">3. Do we have a reason to include geotiff in the mrsid makefile?<br><br></span><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"></span></p>
</div><p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US">Yes – though the reason is obscure. It is to build the mrsid driver as a plugin when linking against the MrSID SDK statically (e.g., older SDKs). Most of the GDAL symbols referenced from the MrSID SDK are satisfied by gdal18.dll on the plugin link line, but two symbols that it needs are not exported from gdal18.dll, namely _TIFFmemcpy and __geotiff_size. The alternative to bringing in these symbols manually from the GDAL build or configured TIFF/GeoTIFF would be to link in gdal.lib statically, but that seemed heavy-handed. If you have any better ideas, I am open to suggestion.</span></p>
<div class="im"><p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"></span></p></div></div></div></blockquote><div><br>Kirk,<br><br>I don't think we will forcibly insist on linking to MrSID statically it could probably be phased out in the future versions.<br>
<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div link="blue" vlink="purple" lang="FR"><div><div class="im"><br><p class="MsoNormal" style="margin-left: 35.4pt;">
<span lang="EN-US">3. I would also require the dll names to be detected (MRSID_DLL, LIDAR_DLL)<br><br></span><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"></span></p></div><p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US">Yes I think this is a good addition – I am guessing it is to be able to install those DLLs?</span></p>
<div class="im"><p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"> </span></p><p class="MsoNormal" style="margin-left: 35.4pt;"><span lang="EN-US">It would also be a good thing to detect the dll versions first if MRSID_RDLLBUILD and MRSID_LDLLBUILD are not specified. This doesn't seem to work according to the spec at the beginning of nmake.opt (with a preference for the DLL if both are found)<br>
<br></span><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"></span></p></div><p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US">I will double-check my logic in that section. What specific MrSID DSDK version are you using?</span></p>
<p class="MsoNormal"><span style="font-size: 11pt; color: rgb(31, 73, 125);" lang="EN-US"></span></p></div></div></blockquote><div><br>I'm now woking with a modified version of your nmake.opt locally (thanks again for providing that) so I'm not in sore need of committing this back in GDAL. I'm using this option to compile and create an installer for the mrsid part of GDAL and in this regard I require both the full path to the dll-s and the name of the dll-s (since the names are changing between SDK versions)<br>
<br>I'm compiling binaries with several compiler versions from MSVC2003 up to MSVC2010 (Win32 and Win64). In this regard I'm using the following versions of the SDKs:<br><br>for compiling with gdal 1.8:<br>Unified_DSDK_8.0_win64-vs10<br>
Unified_DSDK_8.0_win32-vs10<br>Unified_DSDK_8.0_win64-vc9<br>Unified_DSDK_8.0_win32-vc9<br>Raster_DSDK_8.0_win64-vc8<br>Raster_DSDK_8.0_win32-vc8<br>Geo_DSDK-7.0.0.2167.win32-vc7<br><br>for compiling with gdal 1.7:<br>Geo_DSDK-7.0.0.2181.win64-vc9<br>
Geo_DSDK-7.0.0.2181.win32-vc9<br>Geo_DSDK-7.0.0.2167.win64-vc8<br>Geo_DSDK-7.0.0.2167.win32-vc8<br>Geo_DSDK-7.0.0.2167.win32-vc7<br><br><br>Best regards,<br><br>Tamas<br></div></div><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>