Since we no longer support armadillo 5 and all major machines are 64bit, we will
stop testing for 32bit unless we are cross compiling. Note that, I am not sure
that modifying this header file is the way to go, since we will never
compile Armadillo.
Signed-off-by: Omar Shrit <shrit@lri.fr>
If the user is compiling their application with OpenMP but mlpack was not
compiled with OpenMP (or vice versa) we issue a warning. This situation can
cause disaster with recent versions of Armadillo.
Previously CMake block termination commands were required to have arguments
matching the command starting the block. This is no longer necessary and not
the preferred style. This type of change has been implemented in other open
source projects such as: ITK, vxl, and BRAINSTools. It is the preferred style
for KDE. Consistent style practices help improve readability and
maintainability of code.
This was implemented with the shell script:
NOTE: MUST USE GNU compliant version of sed
in else endif endforeach endfunction endmacro endwhile; do
echo 's/\b'"$c"'\(\s*\)(.\+)/'"$c"'\1()/'
done >convert.sed \
&& git ls-files -z -- bootstrap '*.cmake' '*.cmake.in' '*CMakeLists.txt' \
| xargs -0 gsed -i -f convert.sed \
&& rm convert.sed
This is a reimplementation of ITK commit
e52dbe7fa476f6283c9b9d1507fca052355113a4
Fixes#585
Omar Shrit