Skip to content

Add installation targets#30

Merged
daxmawal merged 5 commits into
mainfrom
add_install_targets
Jun 19, 2026
Merged

Add installation targets#30
daxmawal merged 5 commits into
mainfrom
add_install_targets

Conversation

@tretre91

@tretre91 tretre91 commented Jun 19, 2026

Copy link
Copy Markdown
Member

This PR introduces installation targets so that the extract/replay libs and the tool can be installed and consumed by other CMake projects.

I also moved all files in src/* to src/kkf/* so that the headers can be included as #include <kkf/replayer.hpp>
The PR also adds CI steps to install the project and use it in a separate CMake project.

@daxmawal daxmawal self-requested a review June 19, 2026 12:29
@daxmawal

daxmawal commented Jun 19, 2026

Copy link
Copy Markdown
Member

You can update the readme with :

#include <kkf/extractor.hpp>
#include <kkf/replayer.hpp>

Also in the readme, maybe an include example, (I don't know if it's necessary) ?

find_package(kkf REQUIRED)

target_link_libraries(my_extractor PRIVATE kkf::kernel_extractor)
target_link_libraries(my_replayer PRIVATE kkf::kernel_replayer)

Comment thread src/kkf/replay/CMakeLists.txt Outdated
Comment thread tests/install_test/CMakeLists.txt
Comment thread src/kkf/replay/CMakeLists.txt Outdated
Comment thread src/kkf/replay/CMakeLists.txt Outdated
@daxmawal

Copy link
Copy Markdown
Member

Thanks @tretre91

@daxmawal daxmawal merged commit 8b03571 into main Jun 19, 2026
6 checks passed
@daxmawal daxmawal deleted the add_install_targets branch June 19, 2026 13:42
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.

2 participants