[cmake] move ExternalProject_Add to subdir for consistency, and define proper GSL,FFTW,TBB targets#21904
[cmake] move ExternalProject_Add to subdir for consistency, and define proper GSL,FFTW,TBB targets#21904ferdymercury wants to merge 35 commits intoroot-project:masterfrom
Conversation
c5e599c to
cb03ce1
Compare
Test Results 19 files 19 suites 2d 16h 37m 47s ⏱️ For more details on these failures, see this check. Results for commit 283614a. ♻️ This comment has been updated with latest results. |
Co-authored-by: ferdymercury <ferdymercury@users.noreply.github.com>
1f27c07 to
283614a
Compare
|
@bellenot could you let me know what is the contents (filenames) inside:
Or alternatively, what targets are defined once you run find_package with the new FindFFTW.cmake script in this branch ? |
|
@ferdymercury here is the content of And I'll try your branch... |
|
@ferdymercury BTW, what is the problem with FFTW? |
|
Here are (some of) the errors on Windows: Hard to tell where they are coming from (i.e, which DLL is not found). I'll have to debug it locally |
The current FindFFTW.cmake did not define a proper target so I borrowed a more up-to-date one by @egpbos but it seems it had a bug for Win32. I just fixed it so that it now finds all variants of FFTW3 correctly on Windows. |
for win32

Try unbloating the huge SearchInstalledSoftware
and use always subdirs for ExternalProject_Add for symmetry with other builtins