Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add debugging with Visual and PDBs page #3694

Merged

Conversation

juansblanco
Copy link
Contributor

@juansblanco juansblanco commented Apr 23, 2024

Add example off debugging shared dependencies with Visual Studio, and explain usage of new hook that copies PDBs to package.

@juansblanco juansblanco changed the title pdbs docu wip Add debugging with Visual and PDBs page Apr 23, 2024
@juansblanco juansblanco marked this pull request as ready for review May 6, 2024 10:16
Comment on lines 10 to 12
compilation of the libraries. When using Conan these PDBs are generated in the build folder, which is generated during
the building of the libraries and it's no longer needed afterwards. For example, cleaning the Conan cache with
``conan cache clean`` will remove the build folder.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
compilation of the libraries. When using Conan these PDBs are generated in the build folder, which is generated during
the building of the libraries and it's no longer needed afterwards. For example, cleaning the Conan cache with
``conan cache clean`` will remove the build folder.
compilation of the libraries. When using Conan these PDBs are generated in the build folder, during the building of the libraries and are no longer needed afterwards. For that reason it's a common operation to clean the Conan cache with
``conan cache clean`` to remove the build folder and save disk space.

Comment on lines 7 to 8
We previously discussed how to debug dependencies in Visual Studio, but when creating your project with Conan is
possible that the original build folder and build files don't exist. Conan packages don't contain the necessary
Copy link
Contributor

Choose a reason for hiding this comment

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

what does "creating your project with Conan" mean?

the building of the libraries and it's no longer needed afterwards. For example, cleaning the Conan cache with
``conan cache clean`` will remove the build folder.

The goal is to store the necessary PDB files in the package folder to make sure they are always present and don't depend on the
Copy link
Contributor

Choose a reason for hiding this comment

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

This introduction needs to be better linked with the previous paragraph.

Comment on lines +64 to +66
doesn't have any information on the dependencies to debug.


Copy link
Contributor

Choose a reason for hiding this comment

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

Add a sentence to link better with the next section.

Copy link
Contributor

@czoido czoido left a comment

Choose a reason for hiding this comment

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

I changed the target branch to the release one so it's published immediately, good job!

@czoido czoido merged commit bd8de78 into conan-io:release/2.3 May 8, 2024
2 checks passed
franramirez688 pushed a commit to Neeeflix/conan-docs that referenced this pull request Jun 4, 2024
* pdbs docu wip

* improve docu pdbs

* fix

* rewrite

* missing lines

* install hook instructions

* further improve docu

* more fixes

* rewrite to show example

* remove example images

* fixes

* reduce pdb info

* add static libs note

* further improve

* Update examples/dev_flow/debug/debugging_visual.rst

---------

Co-authored-by: Carlos Zoido <mrgalleta@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants