-
Notifications
You must be signed in to change notification settings - Fork 118
Fix compilation on newer compilers #91
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
Conversation
src/native/counters.cc:26:6: error: variable or field '__sanitizer_cov_8bit_counters_init' declared void 26 | void __sanitizer_cov_8bit_counters_init(uint8_t* start, uint8_t* stop); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ src/native/counters.cc:26:41: error: 'uint8_t' was not declared in this scope 26 | void __sanitizer_cov_8bit_counters_init(uint8_t* start, uint8_t* stop); | ^~~~~~~ src/native/counters.cc:24:1: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'? 23 | #include "macros.h" +++ |+#include <cstdint>
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Tobias-Fischer
commented
Sep 2, 2024
Signed
bollwyvl
commented
Sep 2, 2024
I think the changes may go deeper for OSX, as seen on: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1016883&view=logs&jobId=9c5ef928-2cd6-52e5-dbe6-9d173a7d951b&j=9c5ef928-2cd6-52e5-dbe6-9d173a7d951b&t=20c71c51-4b27-578b-485d-06ade2de1d00
Uh oh!
There was an error while loading. Please reload this page.