NVIDIA / gdrcopy

A fast GPU memory copy library based on NVIDIA GPUDirect RDMA technology
MIT License
898 stars 144 forks source link

Remove check header dependency + include string #280

Closed ivankochin closed 1 year ago

ivankochin commented 1 year ago

If I understand correctly check lib dependency was removed several month ago, but common.hpp file still contains header inclusion. When I removed this header, I got the following errors on Ubuntu 20.04:

image

So I decided to fix two problems by one patch.

drossetti commented 1 year ago

@ivankochin Thank you for this patch. FYI we had a similar fix in our pipeline which has been merged via 8f61796c9596. So this is not needed anymore.