-
Notifications
You must be signed in to change notification settings - Fork 423
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
WireGuardKitC: fix build with Xcode 16b1 #33
base: master
Are you sure you want to change the base?
Conversation
3dcdbe7
to
6452cd4
Compare
This commit fixes build errors of the WireGuardKitC target appearing with Xcode 16b1. Errors are: > Declaration of 'u_int32_t'/'u_char'/'u_int16_t' must be imported from module 'DarwinFoundation.unsigned_types.*' before it is required" Signed-off-by: Adam McNight <[email protected]>
6452cd4
to
4fc08d6
Compare
This one is an important commit, merge this to resolve the Xcode 16 issue. |
@zx2c4 Please merge this, simple a one-liner but now the APP won't build on xcode 16 |
Please merge this PR. Really helpful. |
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.
Can confirm that it's working with Xcode 16 (stable version)
related StackOverflow discussion: https://stackoverflow.com/questions/79002585/wireguardkitc-h-failing-to-compile-in-xcode-16-0 |
Please merge the PR. This change is needed. |
Kindly merge it ASAP. This change is required |
@McNight Is there any way we can use your changes using package? |
You can fork my repo and use the fork URL as your dependency in your Package.swift or in Xcode. |
I tried using it but "master" branch is missing changes and "fix_xcode16_build" branch is not working for me. Unable to add the package using that branch. |
anyone solved this? i have the same problem with new xcode |
This commit fixes build errors of the WireGuardKitC target appearing with Xcode 16b1. Errors are: