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

[package] All debug MSVC packages miss debug information aka PDB's #1741

Closed
madebr opened this issue May 27, 2020 · 2 comments
Closed

[package] All debug MSVC packages miss debug information aka PDB's #1741

madebr opened this issue May 27, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@madebr
Copy link
Contributor

madebr commented May 27, 2020

I'm currently trying to track a bug/crash/feature/my-own-fault originating in the systemc package.
Debugging on MSVC can be done using debug information built-in or external PDB's (/Z7 vs /Zi).
The Debug MSVC packages currently do not contain this information.
This means the debug packages do not provide any extra convenience (apart from some extra assertions).

To track the code flow inside the systemc library,
I will have to modify the systemc library to also package the pdb file in debug mode.
I don't think this is the supposed way.

Package and Environment Details (include every applicable attribute)

  • Package Name/Version: all/all
  • Operating System+version: Windows
  • Compiler+version: Visual Studio
  • Docker image: n/a
  • Conan version: conan 1.25.2
  • Python version: Python 3.8.3
@madebr madebr added the bug Something isn't working label May 27, 2020
@jeremy-coulon
Copy link
Contributor

jeremy-coulon commented May 29, 2020

I agree with @madebr .
I mostly use Debug/RelWithDebInfo build types.
Unfortunately I had to change almost all recipes that I use in order to have pdb symbols.

I understand that pdb consumes a lot of disk space and that conan-center-index may be limited by that. If this is the only reason, then I believe that it would be better if conan-center-index only provided Release pre-built binaries and users could add --build missing when they needed another build type.

Of course it would also be nice if conan-center-index CI would try to build for all build types in order to make sure that it works before uploading Release binaries to a conan server.

@madebr
Copy link
Contributor Author

madebr commented Jun 21, 2020

Closing this one. See #1982

@madebr madebr closed this as completed Jun 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants