-
Notifications
You must be signed in to change notification settings - Fork 0
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
ArenaSDK dependency in CMake #38
Conversation
Only supported for arm64 or x86 Linux. The ArenaSDK is pulled from our Google Drive since LUCID blocks the SDK behind a login. Once the SDK is downloaded and extracted, CMake adds the necessary include directories and links the relevant libraries. Also modified the json library's CMakeLists.txt file to move library linking to the deps folder. Running target_link_library from a subdirectory requires CMake 3.13, hence the minimum version has been updated. See the section that says "New in version 3.13": https://cmake.org/cmake/help/latest/command/target_link_libraries.html#overview
For some reason, the JSON library's include directory doesn't show up and is manually specified when the lint target is defined. This could be because the JSON project adds it's include directory as an INTERFACE instead of PRIVATE. Therefore, the include directories aren't available by getting the library's INCLUDE_DIRECTORIES target property. https://cmake.org/cmake/help/latest/command/target_include_directories.html
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.
Works
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.
Successfully got the arena dep.
- can you double check you have gitignored all the new arena stuff?
- NixOS actually doesn't have
/etc/ld.so.conf.d/
so I need to something special for that idk yet. Probably will just use the Docker so don't let this block the PR.
All the stuff is downloaded to build/_deps/arena-sdk
I removed the line that modifies the |
Closes #37
Only supported for arm64 or x86 Linux.
The ArenaSDK is pulled from our Google Drive since LUCID blocks the SDK behind a login.
Once the SDK is downloaded and extracted, CMake adds the necessary include directories and links the relevant libraries.
A macro called "ARENA_SDK_INSTALLED" is enabled if the host is Linux and was able to install the Arena dependencies.
Also modified the json library's CMakeLists.txt file to move library linking to the deps folder.
Running target_link_library from a subdirectory requires CMake 3.13, hence the minimum version has been updated. See the section that says "New in version 3.13": https://cmake.org/cmake/help/latest/command/target_link_libraries.html#overview