Skip to content

Conversation

@akien-mga
Copy link
Member

Trust me, Eldritch horrors lie in wait here.
Let's go back to the previous brittle setup that worked.

Fixes #106376.

Also clean up some dead code.

Trust me, Eldritch horrors lie in wait here.
Let's go back to the previous brittle setup that worked.

Fixes godotengine#106376.

Also clean up some dead code.
@akien-mga
Copy link
Member Author

Oh well it brings back a truckload of warnings :(

@akien-mga
Copy link
Member Author

Just pushed a complete shot in the dark, CC @TCROC.

We might just have to give up and bring back the bazillion warning ignore pragmas removed in #104893.

@TCROC
Copy link
Contributor

TCROC commented May 14, 2025

I put in a PR over here: #106416. This is the solution we used in our fork. Hopefully this fixes it without bringing back the warnings. I will test out this PR to confirm whether or not it fixes things for us as well

@TCROC
Copy link
Contributor

TCROC commented May 14, 2025

@akien-mga I tested it out... unfortunately the shot in the dark misses / doesn't work :(

I tried both Append and Prepend. The only thing that works is reverting to CPPPATH :/

@Repiteo
Copy link
Contributor

Repiteo commented Sep 18, 2025

Superseded by #109749

@Repiteo Repiteo closed this Sep 18, 2025
@Repiteo Repiteo removed this from the 4.5 milestone Sep 18, 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.

Can no longer build godot and godot-nir-static from source together

3 participants