-
Notifications
You must be signed in to change notification settings - Fork 539
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
Generate debug info when building with MSVC #178
Conversation
@jethrogb, thanks for your PR! By analyzing the history of the files in this pull request, we identified @alamaison, @wildart and @dbyron0 to be potential reviewers. |
@alamaison @wildart @dbyron0 ping |
Sorry, I imagine this is right but know very little about cmake. |
@alamaison @wildart @dbyron0 ping |
Don't you just need to switch to the Debug or RelWithDebugInfo configuration in the generated visual studio project? |
It's been a long time. I think I tried that but it didn't work. Do you want me to check again? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Instead of marking it stale, maybe just merge it? |
Anyone (else) building on Windows who can say something 👍 or 👎 about this PR? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
not stale |
@jethrogb did you happen to test the suggestion from @alamaison about changing the project configuration? |
Ok I finally booted into Windows to test this flow again. I'm not using the graphical Visual Studio interface to build anything. I'm just using CMake for the entire build (with the Visual Studio backend). I confirmed that it still doesn't generate symbols without this change. |
I'm not sure why the Visual Studio generators wouldn't do this automatically. I copied what libgit2 is doing.