search

FFTW / fftw3

DO NOT CHECK OUT THESE FILES FROM GITHUB UNLESS YOU KNOW WHAT YOU ARE DOING. (See below.)
GNU General Public License v2.0
2.66k stars 651 forks source link

Update FFTW3Config.cmake.in #346

Closed perache closed 3 months ago

perache commented 5 months ago

The file FFTW3LibraryDepends.cmake is missing. Workaround to have working find_package

Fixes FFTW/fftw3/issues/130 as proposed by @sergiomb2 and @ferdnyc

sergiomb2 commented 5 months ago

please squash the commits in one commit thank you

perache commented 5 months ago

Done, sorry for the mistake.

perache commented 3 months ago

Is there any missing action on my side.

sergiomb2 commented 3 months ago

For me it is approved, we are already using it on Fedora https://src.fedoraproject.org/rpms/fftw/tree/0342b8929c3a1bda8565a99c553e91d450206d71

ferdnyc commented 3 months ago

Also worth noting that #338 contains a perfectly-valid fix for this as well, using include( ... OPTIONAL). (A feature I too was unaware of until just now.)