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

CMake: XCode dependency chain fixes #1730

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

enetheru
Copy link
Collaborator

@enetheru enetheru commented Mar 6, 2025

This has been rebased onto #1733

I was working on the github CI and XCode was failing to compile due to the target dependency chain not meeting their requirements.

Turns out it has other implications for dependency generation that I found when compiling a real extension.

This PR brings the dependency chain inline with XCode's expectations, and simplifies some things.

@enetheru enetheru added the cmake label Mar 10, 2025
@enetheru enetheru requested a review from a team as a code owner March 11, 2025 02:27
@enetheru enetheru added the bug This has been identified as a bug label Mar 11, 2025
Add GODOTCPP_TARGET configuration option
Remove loop to generate the godot-cpp.<target> CMake Targets

Rename test bindings target
Update documentation
When attempting to generate XCode projects it would fail due to the target dependency chain not meeting expectations.

This PR, adds the required dependency infomation so that the XCode generator works.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This has been identified as a bug cmake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants