Let target_embed_source inline local header files

nlesc-recruit / cudawrappers

C++ wrapper for the Nvidia C libraries (e.g. CUDA driver, nvrtc, cuFFT etc.)

https://cudawrappers.readthedocs.io/en/latest/

Apache License 2.0

5 stars 5 forks source link

Let target_embed_source inline local header files #263

Closed csbnw closed 7 months ago

csbnw commented 7 months ago

Description

target_embed_source will now look for includes of local header files and inline these before embedding the source file.

Related issues:

https://github.com/nlesc-recruit/cudawrappers/issues/262

Instructions to review the pull request

Check that CHANGELOG.md has been updated if necessary

csbnw commented 7 months ago

Nice! The CI is failing though - can't find libcuda.so anymore. I'm not sure that's a a result of the changes in this PR or something else...

I noticed as well, I don't know what happened there. There is nothing in this PR that could affect that part. I did notice that the linter preferred different variable names in the new CMake code, so I changed it.

csbnw commented 7 months ago

I will proceed with merging this PR, the CI can be fixed in another PR.