-
Notifications
You must be signed in to change notification settings - Fork 9
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
Use the same Zig version for darwin #46
Comments
I understand the version of Zig to be very fragile and the specifics were picked for toolchains that worked in testing. That was a while ago though. |
Indeed. I'm trying to make sure that all of the specific (and poorly documented) workarounds are documented somewhere. This doesn't necessarily mean that we need to remove them right now but we need to be aware of why they exist as well as how and when they can be removed. Like you say, this was a while ago and without an issue to track it and with poor code documentation, it would unfortunately just have rotted away in the codebase. |
Unfortunately we cannot move to a newer version yet. See fyne-io/fyne-cross#276 |
If something isn't working upstream then we should open a bug there otherwise we can't be certain that it is ever going to be fixed. |
The macOS images (darwin ) use a different version of Zig but does not motivate exactly whey. If this is fixed with newer versions, we should try to move to using the same one for all targets to avoid code complexity.
See
fyne-cross-images/darwin/Dockerfile
Line 9 in 25ee723
Potentially related to #21.
The text was updated successfully, but these errors were encountered: