-
Notifications
You must be signed in to change notification settings - Fork 26
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 .github/workflows/build.yml #32
Conversation
6db80df
to
d6800f1
Compare
51d633b
to
0e50599
Compare
0e50599
to
1bf35f9
Compare
Just updated the CI setup to use the latest (Dec 13) official swift.org "main" nightly toolchain, which has the two C++ interop fixes that were blocking the build (swiftlang/swift#78100, swiftlang/swift#78147). The CI is green, the build passes now 🥳 Resolves a bunch reported issues: |
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.
Please give this a nice commit message when merging!
@@ -38,6 +38,8 @@ | |||
// | |||
// connectedhomeip/src/credentials/FabricTable.h:82:69: error: use of undeclared identifier 'strnlen' | |||
extern "C" size_t strnlen(const char *s, size_t maxlen); | |||
// esp-matter/components/esp_matter/esp_matter_client.h:57:26: error: use of undeclared identifier 'strdup' | |||
extern "C" char *strdup(const char *s1); |
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.
Is there a reason to do this instead of #include <string.h>
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.
I didn't spend time investigating why, but #include <string.h>
didn't resolve the problem.
No description provided.