Opened 4 years ago

Closed 4 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 4 years ago.
Patch to output pdb file for normal Windows build
nmake_with_pdb_option.patch (2.2 KB) - added by tolon 4 years ago.
Patch for optional pdb output Windows release build

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by tolon

Attachment: output_windows_pdb.patch added

Patch to output pdb file for normal Windows build

comment:1 Changed 4 years ago by tolon

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

comment:2 Changed 4 years ago by Even Rouault

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.

Changed 4 years ago by tolon

Attachment: nmake_with_pdb_option.patch added

Patch for optional pdb output Windows release build

comment:3 Changed 4 years ago by tolon

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 Changed 4 years ago by Even Rouault

Milestone: 1.11.0
Resolution: fixed
Status: newclosed

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.