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

Build otherlibs with debug info in C stubs #9804

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

stedolan
Copy link
Contributor

The Unix library and other otherlibs/ denizens don't get -g on their C stubs, which makes debugging annoying, so this patch adds it. (I copy-pasted the Makefile gunk from runtime/)

@shindere
Copy link
Contributor

shindere commented Jul 27, 2020 via email

@dra27
Copy link
Member

dra27 commented Jul 27, 2020

otherlibs/systhreads/Makefile I think can use it, too (it doesn't use Makefile.otherlibs.common).

@stedolan
Copy link
Contributor Author

otherlibs/systhreads/Makefile I think can use it, too (it doesn't use Makefile.otherlibs.common).

Thanks, done.

@dra27
Copy link
Member

dra27 commented Jul 28, 2020

Ugh, sorry to come back with another nit, but I do think this warrants a Changes entry, just because some obscure user or packager may notice it!

@stedolan
Copy link
Contributor Author

Changes changes changed.

Copy link
Member

@dra27 dra27 left a comment

Choose a reason for hiding this comment

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

Lovely! Good to go when CI catches up...

@dra27 dra27 merged commit a7ecba9 into ocaml:trunk Jul 29, 2020
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