Kirk,

I've finally managed to get my builds fixed. I wanted to rely on your
detection logic in nmake.opt but it seems I'd require some minor
modifications (the patch attached):

1. I've found a syntax error in the line: !ELSE IF EXIST("$(R700)"), It may
be due to the content of $(R700) probably.
2. Auto detection of MRSID_JP2 is not working. Need to comment out
"MRSID_JP2 =  NO" at the beginning
3. I would also require the dll names to be detected (MRSID_DLL, LIDAR_DLL)


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)

This detection logic appears to be a cool addition in GDAL, though.

Best regards,

Tamas





2011/1/11 Tamas Szekeres <[email protected]>

>
>
> 2011/1/11 Kirk McKelvey <[email protected]>
>
>
>> Tamas, from what I understand you have tweaked the layout of the MrSID SDK
>> used in your daily windows builds.  Depending on the details this might work
>> for you anyway; if it doesn’t, let me know.
>>
>>
> Kirk,
>
> The changed were breaking, I'm about to fix that in some way. I would have
> the following questions:
>
> 1. Do we officially drop support for the older dsdk-s for example:
> Geo_DSDK-7.0.0.2167.win32-vc7 which use a different directory layout as
> expected by the new nmake.opt. For example the recent SDK is no available
> for MSVC2003 and therefore we should probably use this older version.
>
> 2. Am I right that all the magic in the new nmake.opt is related how to set
> MSRID_LIB, MRSID_INCLUDE and MRSID_FLAGS dynamically which I could also set
> manually by not specifying MRSID_DIR in the builds?
>
> 3. Do we have a reason to include geotiff in the mrsid makefile?
>
> Best regards,
>
> Tamas
>
>
>

Attachment: nmake.opt.patch
Description: Binary data

_______________________________________________
gdal-dev mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/gdal-dev

Reply via email to