Opened 10 years ago

Closed 10 years ago

#5420 closed enhancement (fixed)

Add pdb output to normal Windows build

Reported by: tolon Owned by: warmerdam
Priority: normal Milestone: 1.11.0
Component: default Version: unspecified
Severity: normal Keywords: windows build
Cc:

Description

The current build scripts are not configured to output a pdb file when building on Windows. This file is very useful if investigating problems later on, e.g. when using a debugger or analyzing a crash dump.

Attachments (2)

output_windows_pdb.patch​ (922 bytes ) - added by tolon 10 years ago.
Patch to output pdb file for normal Windows build
nmake_with_pdb_option.patch​ (2.2 KB ) - added by tolon 10 years ago.
Patch for optional pdb output Windows release build

Download all attachments as: .zip

Change History (6)

by tolon, 10 years ago

Attachment: output_windows_pdb.patch​ added

Patch to output pdb file for normal Windows build

comment:1 by tolon, 10 years ago

Effect on build time tested: before change, ~16 mins; after change, ~18 mins.

comment:2 by Even Rouault, 10 years ago

I'm not sure we want to do generate .pdb unconditionnaly on release builds. Perhaps add an option WITH_PDB ? People would do "nmake -f makefile.vc WITH_PDB=1" if they want a release build with pdb generation.

by tolon, 10 years ago

Attachment: nmake_with_pdb_option.patch​ added

Patch for optional pdb output Windows release build

comment:3 by tolon, 10 years ago

PDBs are very useful build artifacts on Windows for doing crash dump analysis, but I'm happy to compromise and make them optional (defaulting to off for the normal release build). New patch attached - attachment:nmake_with_pdb_option.patch​

comment:4 by Even Rouault, 10 years ago

Milestone: → 1.11.0
Resolution: → fixed
Status: new → closed

trunk r27143 "nmake.opt: add WITH_PDB=1 option to optionaly generate .pdb file on Release builds (patch by tolon, #5420)"

Note: See TracTickets for help on using tickets.