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

Adding CMakeLists.txt to enable consuming Coyote as a library #108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sven-weber
Copy link

Description

As part of a master thesis, we are currently using Coyote to implement FPGA-based database operators. These operators will be integrated into the code base of an existing database. Therefore, we need to consume Coyote as a library from another C++ code base. This PR adds the CMake configurations that are required to consume Coyote as a library, because they have been missing up until now.

Once these changes are merged, one can build the /sw folder of Coyote as a library as follows:

git clone https://github.com/sven-weber/Coyote
pushd coyote/sw
pushd build
cmake_options=(
  "-DCMAKE_BUILD_TYPE=$build_type"
  "-DCMAKE_INSTALL_PREFIX=$dest"
  "-DCOYOTE_VERBOSE=OFF"
)
cmake "${cmake_options[@]}" ..

The majority of the CMakeLists.txt file has been copied from the existing CMakeLists.txt inside /cmake/FindCoyoteSW.cmake.

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • A new research paper code implementation
  • Other

Tests & Results

I tried to consume Coyote as a library with the steps described above and the changes proposed in this PR. This worked without problems!

Checklist

  • [N/A] I have commented my code and made corresponding changes to the documentation.
  • [N/A] I have added tests/results that prove my fix is effective or that my feature works.
  • My changes generate no new warnings or errors & all tests successfully pass.

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.

1 participant