-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Update SConstruct and .gdextension file of the godot-cpp example with newest conventions
#11405
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
base: master
Are you sure you want to change the base?
Update SConstruct and .gdextension file of the godot-cpp example with newest conventions
#11405
Conversation
47c8295 to
edf4390
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nitpick, otherwise it looks pretty good 👍🏻
edf4390 to
bf09cfd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍🏻
|
@Ivorforce Afraid this needs a rebase, but otherwise looks good to go for me :) |
…with newest conventions. # Conflicts: # tutorials/scripting/cpp/gdextension_cpp_example.rst
bf09cfd to
c46a399
Compare
|
Looks like #11381 broke the example since the example |
The changes:
SConstructfile currently lists properties. I don't think this is very useful, because a non-SCons user will not understand how to use this information anyway. Instead, I add a link to the SCons docs.SConstructfile..frameworklogic fromSConstructand.gdextensionfile. We've figured some months that this wasn't needed (matching godot-cpp-template)SHLIBPREFIXalong withSHLIBSUFFIX, to removelibprefix from DLL files (matching godot-cpp-template)..gdextensionfiles. This makes it possible to rename the folders they are in, leading to fewer problems. It also solves an unfixed hot reload problem on macOS.dependenciessection since it's no longer necessary, add a link to the.gdextensionfile docs instead.