-
Notifications
You must be signed in to change notification settings - Fork 19
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
No mention of linking TBB libraries in "Making things parallel" #32
Comments
Is there anything mentioning installing the library too? I had to find the repo for ubuntu (which turned out to be libtbb-dev), but I couldn't see it mentioned anywhere - that might also be useful since some people won't have used the linux package manager much. |
Yes - that is under "Choose a platform" -> "Installing libraries". At least in the Linux platform description, haven't bothered checking any other ones. Different distros will have different TBB package names, but the Ubuntu/Debian one is actually listed. For Arch it was intel-tbb. |
That looks like a regression on my part, as I don't think it was a problem last time. The reason I'll update the instructions, as it is definitely a non-obvious step for most people and should Thanks. |
(Leaving the issue open for a while, so that people can see it.) |
I'm working through the guide right now, and I got slightly stuck on the "Making things parallel" part with the compiler spitting out a large list of tbb-related errors.
Then, with help of Google, I realised I'm not importing the TBB libraries into the compiler. A quick look in the makefile confirmed my suspicion - "LDLIBS += -ltbb" is commented out by default, and there's no mention of this in the tutorial.
I feel like it would be a good idea to add a sentence reminding us to go back and update the makefile before trying to recompile for TBB, unless this is a deliberate attempt to make people learn how to debug (i.e. Google the errors).
The text was updated successfully, but these errors were encountered: