Skip to content

Conversation

@Spartan322
Copy link
Contributor

No description provided.

enetheru and others added 11 commits March 17, 2025 11:26
People seem to want to build godot-cpp without any arguments, not even specifying a target.

We have to detect if we are top level though because it conflicts with being consumed.

(cherry picked from commit 8b33e73)
The SCons build doesn't do this, so neither should CMake
Excpetions are already handled by DISABLE_EXCEPTIONS option

(cherry picked from commit 43690cb)
Add GODOTCPP_TARGET configuration option
Remove loop to generate the godot-cpp.<target> CMake Targets

Rename test bindings target
Update documentation

(cherry picked from commit 89abe15)
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.

(cherry picked from commit f83f364)
It should be ok when folders exist. Exception shouldn't be thrown.

Update binding_generator.py

It should be ok when folds exist. It will fail to build without this patch,  in vs code on windows with compiler ( visual studio community 2022 amd 64) .

Co-Authored-By: Chris Cranford <[email protected]>
(cherry picked from commit d79959c)
Made all warnings private.
Warning as error private
Consistency in  generator expressions

(cherry picked from commit 5eb16d0)
Cherry-picks for the godot-cpp 4.4 branch - 1st batch
@Spartan322 Spartan322 merged commit 70426d7 into Redot-Engine:4.4 Apr 22, 2025
11 checks passed
@Spartan322 Spartan322 deleted the 4.4-merge/6388e26 branch April 22, 2025 22:58
Spartan322 added a commit to Spartan322/redot-cpp that referenced this pull request Apr 23, 2025
@Spartan322 Spartan322 restored the 4.4-merge/6388e26 branch April 23, 2025 06:03
@Spartan322 Spartan322 changed the title [4.4] Merge commit godotengine/godot@6388e26 [4.4] Merge commit godotengine/godot-cpp@6388e26 Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants