Skip to content

Help IDEs resolve gen/include immediately#118

Merged
Ivorforce merged 1 commit into
godotengine:mainfrom
van800:main
May 11, 2026
Merged

Help IDEs resolve gen/include immediately#118
Ivorforce merged 1 commit into
godotengine:mainfrom
van800:main

Conversation

@van800
Copy link
Copy Markdown
Contributor

@van800 van800 commented May 6, 2026

Create the godot-cpp/gen/include directory at configure time so IDEs can see generated headers immediately after generate_bindings target produces them.

The problematic use-case, which I have fixed for Rider, CLion and VS:

  1. clone a fresh copy of godot-cpp-template
  2. open example_class.cpp
    observe red code
  3. call generate_bindings target
    still observe red code, while expected result would be to see all resolved.

Notes:

  • maybe this should be done somewhere in the godot-cpp, not sure.
  • It would be even nicer to call the generate_bindings target in the design time, but that would be arguable bigger change.

…can see generated headers immediately after generate_bindings target produces them
@van800
Copy link
Copy Markdown
Contributor Author

van800 commented May 6, 2026

@enetheru, what do you think?

@enetheru
Copy link
Copy Markdown
Contributor

enetheru commented May 6, 2026

is there changes missing? how does creating that directory effect anything?

@van800
Copy link
Copy Markdown
Contributor Author

van800 commented May 6, 2026

I suppose, if folder does not exist, IDE can't watch it for further changes. I haven't dug too deep, but I checked the behavior is consistent with Clion, Rider and VS.

@enetheru
Copy link
Copy Markdown
Contributor

enetheru commented May 6, 2026

I suppose, if folder does not exist, IDE can't watch it for further changes. I haven't dug too deep, but I checked the behavior is consistent with Clion, Rider and VS.

well if it helps i'm all for it.

@Ivorforce
Copy link
Copy Markdown
Member

@enetheru If you are happy with it, please approve and we'll merge.

@enetheru
Copy link
Copy Markdown
Contributor

@enetheru If you are happy with it, please approve and we'll merge.

I'm not a member of the template, i cant approve

@Ivorforce
Copy link
Copy Markdown
Member

You can always approve, it just won't be green.

Copy link
Copy Markdown
Contributor

@enetheru enetheru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Ivorforce Ivorforce merged commit 7747217 into godotengine:main May 11, 2026
5 checks passed
@Ivorforce
Copy link
Copy Markdown
Member

Thanks!

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.

3 participants