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 CMAKE_BUILD_TYPE and change a CMAKE flag #43

Closed
wants to merge 9 commits into from

Conversation

lmburns
Copy link
Contributor

@lmburns lmburns commented Sep 2, 2022

What the title says. This is in addition to your pull request already. I tested it and it works just fine

@MordechaiHadad
Copy link
Owner

Hello thanks for the PR! what exactly is RelWithDebInfo for? and could you change from pulling to master to the PR branch?

@MordechaiHadad
Copy link
Owner

Hello thanks for the PR! what exactly is RelWithDebInfo for? and could you change from pulling to master to the PR branch?

@lmburns

@lmburns
Copy link
Contributor Author

lmburns commented Sep 5, 2022

Okay, I will close this PR and open a new one. I thought that I had correctly done the branches firstly, but I guess I didn't. I think I did now.

It leaves debug symbols within the code as far as I know. It is what I have been using to build Neovim whenever I build it from source. It can be found in their wiki as well here.

@lmburns lmburns closed this Sep 5, 2022
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.

None yet

2 participants