Skip to content

Update cmake to 3.10 and add CUDA language support#4619

Merged
kunaltyagi merged 14 commits intoPointCloudLibrary:masterfrom
larshg:UpdateCmake
Aug 5, 2021
Merged

Update cmake to 3.10 and add CUDA language support#4619
kunaltyagi merged 14 commits intoPointCloudLibrary:masterfrom
larshg:UpdateCmake

Conversation

@larshg
Copy link
Copy Markdown
Contributor

@larshg larshg commented Feb 21, 2021

Bump to 3.10
Use CUDA first language support.

@larshg larshg force-pushed the UpdateCmake branch 18 times, most recently from 8090952 to ee10594 Compare February 26, 2021 23:24
Copy link
Copy Markdown
Member

@kunaltyagi kunaltyagi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Partial review

Comment thread CMakeLists.txt Outdated
Comment thread CMakeLists.txt Outdated
Comment thread cmake/Modules/FindOpenMP.cmake
Comment thread cmake/pcl_find_cuda.cmake Outdated
Comment thread cmake/pcl_find_cuda.cmake Outdated
Comment thread cuda/apps/CMakeLists.txt Outdated
@larshg
Copy link
Copy Markdown
Contributor Author

larshg commented Feb 27, 2021

@kunaltyagi do you have any idea why 18.04 fail to find cuda.h file?

Currently booting up a virtual to do some more intense debugging locally.

@kunaltyagi
Copy link
Copy Markdown
Member

Sorry, not on top of my head.

@larshg larshg force-pushed the UpdateCmake branch 9 times, most recently from b618316 to f9a8229 Compare February 28, 2021 15:27
@BaltashovIlia
Copy link
Copy Markdown

Hi, @mvieth, could you review this MR?

Copy link
Copy Markdown
Member

@mvieth mvieth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, as far as my CMake and CUDA knowledge goes 😄

@kunaltyagi
Copy link
Copy Markdown
Member

@larshg merge?

@larshg
Copy link
Copy Markdown
Contributor Author

larshg commented Aug 5, 2021

Let it fly 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: enhancement Meta-information for changelog generation module: cmake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants