change FindNumPy.cmake for FindNumpy.cmake

[marnunez]

No description provided.

[Alzathar]

Can you detail this issue? Is it a CMake detection problem? Or the usage of the associated macro in the CMake scripts?

[marnunez]

Sorry about this, I accidentally sent the ticket without description. My version of CMake does not find the NUMPY_DIR, keeps asking for FindNumpy.cmake. Which in fact does not exists, it's FindNumPy.cmake. So I guess it's either change the file name or modify the makefile, right? El 25 ene. 2017 11:05 AM, "Alzathar" <notifications@github.com> escribió:

…

Can you detail this issue? Is it a CMake detection problem? Or the usage of the associated macro in the CMake scripts? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABZwWO9LKuVRSewViALxzMTp8TEpgNgVks5rV1ZRgaJpZM4LtgEX> .

[marnunez]

cmake 3.7.1
Linux 4.8.0-2-amd64 #1 SMP Debian 4.8.15-2 (2017-01-04) x86_64 GNU/Linux

Changing the file name works for me, everything compiles and tests are fine.

[Alzathar]

Can you test on your side the modification of the line openma/cmake/OpenMALanguageBindings.cmake:71.

Instead of FIND_PACKAGE(Numpy REQUIRED) it should be FIND_PACKAGE(NumPy REQUIRED). You might need to clear the CMake cache to see the effect of this modification.

[marnunez]

There, it works. As previously, all tests run successfully except #57: openma_io_handlerplugin_reader_bsf , but I know you're working on it.

[Alzathar]

Thanks for feedback, I will modify the code ASAP.

Regarding the unit test openma_io_handlerplugin_reader_bsf, you should update your openma-data repository. I pushed the missing file yesterday night (see OpenMA/openma-data@9d783f7).