[Liblas-devel] /GR and /Zi in nmake.opt

[Erik Jacobsen]

Hello Howard,

With respect to /GR, I appreciate the explanation for why it is the way 
it is.  I'll live with it, but you might want to put a comment in 
nmake.opt stating that you are using a nonstandard calling convention 
and removing /GR should be the first thing tried to resolve linker errors.

/Z7 predates /Zi (.pdb) so it will be available in compilers prior to 
MSVC 7.1.  MSDN actually suggests using it for libraries:

http://msdn.microsoft.com/en-us/library/958x11bc(VS.80).aspx



Erik


On 9/25/2008 13:19, Howard Butler wrote:
>
> On Sep 25, 2008, at 1:06 PM, Erik Jacobsen wrote:
>
>>
>> Hello Mateusz,
>>
>> Thank you for looking at that so quickly.  As long as I am making 
>> suggestions, I have two more both relating to building with MSVC 
>> version 9, SP1 using makefile.vc and nmake.opt:
>>
>> 1) Please consider removing /GR from OPTFLAGS.  Using a non-default 
>> calling convention produces somewhat puzzling linker errors when 
>> attempting to statically link against libLas.
>
> IIRC, I needed /GR to be able to link libLAS against the OSGeo4W GDAL 
> and GeoTIFF DLLs.  Static linkage of libLAS is not our typical usage 
> scenario.  Please consider registering for the wiki and creating a 
> page that describes how to build libLAS statically for those 
> interested...  Because I want to be able to easily generate official 
> libLAS windows releases targeted at OSGeo4W using msvc 7.1/8, I don't 
> see us removing /GR.
>
>
>>
>>
>> 2) Please consider using /Z7 for debug info in OPTFLAGS instead of 
>> /Zi.  /Z7 produces C7 style debugging information which is directly 
>> embedded in the library eliminating the need for a separate .pdb file.
>>
>
> Does /Z7 work for msvc 7.1/8 as well?  It would be wonderful to have a 
> -g like option instead of those damn .pdb files everywhere.
>
> Howard
>

-- 
Erik Jacobsen, Power Line Systems, Inc.
Email: jacobsen at powline.com
Phone: (608) 238-2171 x104
Fax:   (608) 238-9241
Web:   http://www.powline.com/