Skip to content
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

several improvements #80

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
8 changes: 7 additions & 1 deletion src/include/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
########### install files ###############

INSTALL_FILES( /include/fann FILES fann.h doublefann.h fann_internal.h floatfann.h fann_data.h fixedfann.h fann_activation.h fann_cascade.h fann_error.h fann_train.h fann_io.h fann_cpp.h )
IF(NOT OPENMP_FOUND OR DISABLE_PARALLEL_FANN)
SET(PARALLEL_INCLUDES "")
ELSE(NOT OPENMP_FOUND OR DISABLE_PARALLEL_FANN)
SET(PARALLEL_INCLUDES parallel_fann.h parallel_fann.hpp)
ENDIF(NOT OPENMP_FOUND OR DISABLE_PARALLEL_FANN)

install (FILES fann.h doublefann.h fann_internal.h floatfann.h fann_data.h fixedfann.h fann_activation.h fann_cascade.h fann_error.h fann_train.h fann_io.h fann_cpp.h fann_data_cpp.h fann_training_data_cpp.h ${PARALLEL_INCLUDES} DESTINATION ${INCLUDE_INSTALL_DIR})

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency's sake, install should be INSTALL here.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This INSTALL command was added in CMake 2.4, which I think is old enough for us to assume that everyone has it. The CMakeLists.txt file itself does not seem to require any minimum CMake version.


You are viewing a condensed version of this merge commit. You can view the full changes here.