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

ArenaSDK dependency in CMake #38

Merged
merged 5 commits into from
Oct 22, 2023
Merged

ArenaSDK dependency in CMake #38

merged 5 commits into from
Oct 22, 2023

Conversation

atar13
Copy link
Member

@atar13 atar13 commented Oct 21, 2023

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

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
@atar13 atar13 marked this pull request as ready for review October 22, 2023 03:08
Copy link
Contributor

@Tyler-Lentz Tyler-Lentz left a comment

Choose a reason for hiding this comment

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

Works

Copy link
Contributor

@Samir-Rashid Samir-Rashid left a 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.

@atar13
Copy link
Member Author

atar13 commented Oct 22, 2023

  • can you double check you have gitignored all the new arena stuff?

All the stuff is downloaded to build/_deps/arena-sdk

  • 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.

I removed the line that modifies the /etc/ld.so.conf.d/. The library is linked via CMake and doesn't modify the system.

@atar13 atar13 merged commit 0895aad into main Oct 22, 2023
1 of 2 checks passed
@atar13 atar13 deleted the chore/arena-sdk branch October 22, 2023 18:18
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.

Add Arena SDK to CMake
3 participants