Skip to content
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

Fix some compilation errors on MacOS due to an old SDK #96

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

amontoison
Copy link
Contributor

#94

@cvanaret cvanaret merged commit 44005b7 into cvanaret:main Nov 16, 2024
3 checks passed
@cvanaret
Copy link
Owner

Thanks @amontoison!
The command rm -rf /opt/${target}/${target}/sys-root/System in the x86_64-apple-darwin14 build fails:

[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Printers/Libraries': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/Tcl.framework/Versions/8.4/Headers/tcl-private': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/Tcl.framework/Versions/8.5/Headers/tcl-private': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/ForceFeedback.framework/Versions/A/Modules': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/ForceFeedback.framework/Versions/A/Headers': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/QTKit.framework/Versions/A/Modules': I/O error
[18:02:50] rm: can't remove '/opt/x86_64-apple-darwin14/x86_64-apple-darwin14/sys-root/System/Library/Frameworks/QTKit.framework/Versions/A/Headers': I/O error
...

@amontoison
Copy link
Contributor Author

amontoison commented Nov 16, 2024

Where do you have this error?
Is it in a CI build?

@cvanaret
Copy link
Owner

I retagged to trigger the release build. Perhaps too soon?

@amontoison
Copy link
Contributor Author

It will not work because you have other compilation errors:
#94 (comment)

@cvanaret
Copy link
Owner

Ah sorry, I overlooked that one. Looks like the good old missing include that no compiler ever complained about (except now).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants