Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Enables generating full PDB for the WSL app. #1036

Merged
merged 1 commit into from
Aug 8, 2022
Merged

Enables generating full PDB for the WSL app. #1036

merged 1 commit into from
Aug 8, 2022

Conversation

CarlosNihelton
Copy link
Contributor

By default debug symbols are not exported for the Flutter application runner. Per instructions found in this comment on the Flutter framework repository, this one-liner enables exporting the debug symbols in a program debug database (.pdb) file.

The WSL launcher will be prepared to collect such artifact as part of the release process, providing us with a better crash analysis capabilities, shall we need it.

Ideally we should also collect those files for the plugins as well (those that generate dlls). Per this comment it seems we lack a way to propagate that option from the app to the required plugins. So, for now, let's focus on the app.

@CarlosNihelton CarlosNihelton marked this pull request as ready for review August 4, 2022 21:45
Copy link
Contributor

@jpnurmi jpnurmi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@CarlosNihelton CarlosNihelton merged commit d4d7a04 into canonical:main Aug 8, 2022
@CarlosNihelton CarlosNihelton deleted the app-pdb-deeng-355 branch August 8, 2022 10:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants