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

the CMakeLists makes it so godot-cpp's output static lib is placed in the wrong directory #598

Closed
SeleDreams opened this issue Aug 21, 2021 · 3 comments
Labels
bug This has been identified as a bug

Comments

@SeleDreams
Copy link
Contributor

I think godot-cpp has a small issue in its CMakeLists, rather than using CMAKE_CURRENT_SOURCE_DIR to get the directory of the current cmakelists, it uses CMAKE_SOURCE_DIR which gets the path of the top level cmakelists
the result is that instead of building the .a in its own directory, it will create a bin directory on the top level source directory with the .a inside which is unwanted behaviour since it clutters the directory

a fix would be to replace CMAKE_SOURCE_DIR by CMAKE_CURRENT_SOURCE_DIR

@Calinou Calinou added the bug This has been identified as a bug label Aug 21, 2021
@enetheru
Copy link
Contributor

This is actually intended behaviour. The goal for the cmake build system is to match scons, as scons puts its binaries in godot-cpp/bin so shall cmake.

A change in this behaviour would require a change in the behaviour of scons aswell.

@enetheru
Copy link
Contributor

This has now been changed in master. The issue was brought up again during merge discussion of #1598
The output of the library artifacts is now "${CMAKE_BINARY_DIR}/bin"

This issue can be closed. @dsnopek

@dsnopek
Copy link
Collaborator

dsnopek commented Nov 25, 2024

Thanks!

@dsnopek dsnopek closed this as completed Nov 25, 2024
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
Projects
None yet
Development

No branches or pull requests

4 participants