-
Notifications
You must be signed in to change notification settings - Fork 499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
python3 setup.py install fails because FFTW3 doesn't support installation via CMake #1058
Comments
Just hit this, thank you for the information, you helped me figure out how to compile. Much appreciated. EDIT: I spoke too soon x_x Second EDIT: This did help me figure out how to compile! Just had a few other things to work through. |
You're welcome. This bug did bother me for a while. Glad it helps. |
In case people find this while searching, what I did in my Dockerfiles to fix this is I straight up deleted the fftw cmake directory: RUN rm -rf /usr/local/lib/cmake/fftw3 # as per https://github.com/FFTW/fftw3/issues/130 This forces cmake to use pkg-config which worked for my case. Other options include compiling fftw using cmake, but because that wasn't documented well and has "missing features" I decided against it. This may be fixed for flashlight in #1060 but it is an upstream fftw bug as the original issue says. |
Bug Description
When creating python bindings for flashlight, we run
python setup.py install
in the directorybindings/python
. Thescript
setup.py
starts a new sub-process:flashlight/bindings/python/setup.py
Lines 99 to 104 in 2261fd9
It runs Cmake to build essential components. The problem is that FFTW3 doesn't support installation via CMake according to the reply in the issue:
and the official documentation
Reproduction Steps
cd ./repos/flashlight/bindings/python/
python ./setup.py install
Platform and Hardware
Additional Context
No additional information
Solution
As we can see from the error message:
CMake could find the file
FFTW3LibraryDepends.cmake
due to the inherent bug of FFTW as mentioned before. However, we did have installed FFTW3 via./configure && make && sudo make install
. Since it shows-- FFTW found
, we can safely ignore the error message and continue compiling the python bindings.Solution: Change
subprocess.check_call
tosubprocess.call
to ignore CMake errors.The text was updated successfully, but these errors were encountered: