Look for Threads dependency in CMake config script

jlblancoc / nanoflann

nanoflann: a C++11 header-only library for Nearest Neighbor (NN) search with KD-trees

Other

2.26k stars 491 forks source link

Look for Threads dependency in CMake config script #226

Closed roehling closed 11 months ago

roehling commented 11 months ago

This is a fix for Debian bug #1058950. Unfortunately, both the autopkgtest (via GTest) and my own use already have a find_package(Threads), so this bug never triggered for me.

jlblancoc commented 11 months ago

Oh, it makes sense, thanks! Would you need a new official release to fix it in Debian or is a d/patch ok?

roehling commented 11 months ago

I already made a -2 Debian release with the PR applied, so there's no hurry. Also, most people (me included) never ran into the bug anyway, and it's trivial to work around with an additional find_package(Threads) before the find_package(nanoflann).