-
-
Notifications
You must be signed in to change notification settings - Fork 419
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
Change stack_depth_t to size_t on OpenBSD #4575
Conversation
Hi @dspx-plcr, The changelog - changed label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do. Release notes are added by creating a uniquely named file in the The basic format of the release notes (using markdown) should be:
Thanks. |
@dspx-plcr for the release notes on this, can you note that it was a change to support compiling on OpenBSD 7.6 and that earlier versions of OpenBSD might no longer compile (or won't compile depending on what is true). Thanks. |
I took a look at another PR for the release notes, let me know if you were after something a little different. The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving with the assumption that CI will pass.
Thanks @dspx-plcr. A most excellent first contribution. |
I'm not sure when OpenBSD made this change, but on OpenBSD 7.6,
backtrace
returns asize_t
. Prior to this change, building pony fails with