Skip to content

fix(windows): Do not install PDB debug files when not building in Debug mode - #10911

Open
claucambra wants to merge 2 commits into
masterfrom
work/pdb-windows
Open

claucambra wants to merge 2 commits into
masterfrom
work/pdb-windows

Conversation

@claucambra

Copy link
Copy Markdown
Collaborator

Summary

We currently install the PDB debug files in the release client; seems like we should only do this for debug builds

Checklist

AI (if applicable)

…ug mode

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
@claucambra claucambra added this to the 34.0.5 milestone Sep 24, 2026
@claucambra claucambra self-assigned this Sep 24, 2026
@claucambra

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0

… running debug build

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>
@sonarqubecloud

Copy link
Copy Markdown

@mgallien mgallien left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if we do not want them in release installer packages, why not use Release build type ?
after all, we do generate PDB to discard them
doing this change would prevent building installer packages with PDB if needed

@claucambra

Copy link
Copy Markdown
Collaborator Author

I think if we do not want them in release installer packages, why not use Release build type ?
after all, we do generate PDB to discard them
doing this change would prevent building installer packages with PDB if needed

In our case, in our build infra, we do grab the PDB files and archive them (not sure to what end). So doing this change we will still build and be able to store these files for our purposes, but not install them for the user

@claucambra claucambra modified the milestones: 34.0.5, 35.0.0 Sep 25, 2026
@mgallien mgallien modified the milestones: 35.0.0, 35.0.1 Sep 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants