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

CMake error: target dependency added via target_include_directories #16

Closed
Robert42 opened this issue Mar 14, 2019 · 8 comments
Closed

Comments

@Robert42
Copy link

In the example's cmake file, there's a line

target_include_directories(${order}-${exname} PRIVATE Vulkan::Vulkan)

To my knowledge, target_include_directories only allows directory paths and not targets as arguments. I needed to move the Vulkan::Vulkan dependency to target_link_libraries in order to be able to compile the samples

@lhog
Copy link
Collaborator

lhog commented Mar 30, 2019

Please confirm you can't compile it now (because I can) and provide your system specs: CMake version, OS, Vulkan SDK version, etc.

@9prady9
Copy link

9prady9 commented Apr 5, 2019

CMake import targets are meant to be used with target_link_libraries unless newer versions added support for using them target_include_directories. Vulkan::Vulkan passed to link libraries call should include the header location just as fine.

@lhog
Copy link
Collaborator

lhog commented Mar 28, 2020

See if #18 good enough

@andy-thomason
Copy link
Owner

Let me know how you get on with the merge...

@FunMiles
Copy link
Contributor

FunMiles commented Apr 6, 2020

The offending line has been removed in the PR for #21. If it does resolve the issue completely, then this issue should be closed.

@FunMiles
Copy link
Contributor

FunMiles commented Apr 7, 2020

Now that the PR for #21 has been merged, I think this issue is also addressed. @Robert42, have you tried it ?

@FunMiles
Copy link
Contributor

Without further comment on this issue, should it be closed? @Robert42 can reopen it if there's still an issue for him.

@lhog
Copy link
Collaborator

lhog commented Nov 22, 2020

Reopen, in case the issue persists

@lhog lhog closed this as completed Nov 22, 2020
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

No branches or pull requests

5 participants