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.

Sign up for GitHub

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

Jump to bottom

Official CMake script? #38

Closed
jdumas opened this issue Aug 18, 2017 · 2 comments
Closed

Official CMake script? #38

jdumas opened this issue Aug 18, 2017 · 2 comments

Comments

@jdumas
Copy link
Contributor

jdumas commented Aug 18, 2017

It seems that TBB now provides CMake scripts for integrating source or binaries into a CMake build, so I wonder if you may want to sync your fork with upstream to reflect those changes? At the very least you could put up a notification on the README.
@wjakob
Copy link
Owner

wjakob commented Aug 18, 2017

Nope -- the current CMake support is severely lacking, see oneapi-src/oneTBB#6.

@wjakob wjakob closed this as completed Aug 18, 2017
@jdumas
Copy link
Contributor Author

jdumas commented Aug 18, 2017

Ok! Your fork being behind the official repository, I thought their latest update might have resolved the need for a separate cmake script. My bad.

@jdumas jdumas changed the title Obsolete? Official CMake script? Aug 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jdumas @wjakob