Include cmake bux fix

[galabovaa]

closes #2250

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.94%. Comparing base (d5099ea) to head (a6f6015).
Report is 7 commits behind head on latest.

Additional details and impacted files

@@           Coverage Diff           @@
##           latest    #2253   +/-   ##
=======================================
  Coverage   78.93%   78.94%           
=======================================
  Files         343      343           
  Lines       83419    83419           
=======================================
+ Hits        65850    65851    +1     
+ Misses      17569    17568    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jajhall]

Thanks!

[Mizux]

one question:
while you move the call containing ${CMAKE_CUDA_PATH}/... to avoid it to be empty and thus having an absolute path

Why you didn't use the idiomatic $<BUILD_INTERFACE:${CMAKE_CUDA_PATH}/...> construction instead ?

i.e. never use target include PUBLIC without the generator expression $<BUILD_INTERFACE:> $<INSTALL_INTERFACE:>...
ref: https://cmake.org/cmake/help/latest/command/target_include_directories.html#creating-relocatable-packages

[galabovaa]

Thank you @Mizux I will add $<BUILD_INTERFACE:> in another PR