TomographicImaging/CCPi-Regularisation-Toolkit

find_package openmp

rijobro opened this issue · 3 comments

rijobro commented

The find_package(OpenMP) should have the REQUIRED flag.

It should also be moved up one level, as it is currently placed in the Core folder, but is also required in the Python folder.

dkazanc commented

Thanks, it has been corrected in this PR #137
It is OK @paskino ?

dkazanc commented

I added the requirement in here. I, however left the requirement inside Core folder.

paskino commented

@dkazanc since you moved find_package(OpendMP REQUIRED) up one directory it is not necessary to call it also in /src/Core