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

Non-CUDA build failure #12

Closed
bforsbe opened this issue Jun 22, 2016 · 2 comments
Closed

Non-CUDA build failure #12

bforsbe opened this issue Jun 22, 2016 · 2 comments

Comments

@bforsbe
Copy link
Contributor

bforsbe commented Jun 22, 2016

Originally reported by: Jon Diprose (Bitbucket: well-jon, GitHub: Unknown)


I think this is similar to issue #3 except for a non-CUDA build. Firstly, src/ml_model.cpp won't build without CUDA - see the attached rescomp-make.log and relion-2.0b-ml_model-no-cuda.patch that makes the problem go away by hiding the cuda_skunks.cuh-requiring code in #ifdef CUDA blocks.
However, it then fails to link because it fails to find -lrelion_gpu_util - see the attached rescomp-make-2.log. Suggested fix is relion-2.0b-CMakeLists-no-cuda.patch, which checks CUDA_FOUND to see if relion_gpu_util should be on the list of target_link_libraries.
After that it needs re-cmake-ing, so a complete rebuild - see rescomp-configure-3.log and rescomp-make-3.log - and builds for me! Hurrah.


@bforsbe
Copy link
Contributor Author

bforsbe commented Jun 22, 2016

Original comment by Bjoern Forsberg (Bitbucket: bforsbe, GitHub: bforsbe):


Thanks for that detailed bug report! Indeed we have accidentally left gpu-code visible to the non-cuda compilation. This will be fixed! I'll resolve the issue when you suggested patch is included. Thanks again!

@bforsbe
Copy link
Contributor Author

bforsbe commented Jun 27, 2016

Original comment by Bjoern Forsberg (Bitbucket: bforsbe, GitHub: bforsbe):


This should be fixed in v2.0.b3 (682c312).

@bforsbe bforsbe closed this as completed Jan 26, 2017
biochem-fan pushed a commit that referenced this issue Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant