ViewVC logotype

Revision 6736

Jump to revision: Previous Next
Author: aellery
Date: Thu Nov 1 23:50:29 2018 UTC (5 months, 3 weeks ago)
Changed paths: 3
Log Message:
Minor bug fixes.

I fixed several errors that were preventing eScripts from compiling (on Debian Stretch) when CUDA was enabled.

A set of problems arose due to a preprocessor directive starting on line 869 of RipleyDomain.cpp that the compiler only includes in the specific case where CUDA is enabled but MPI is not. Specifically:
(i) the switch statement inside this directive did not explicitly handle several values of the enum variables in escript/SolverOptions.h and had no default case. As a result the compiler was bouncing back an error. I have resolved this issue by adding a default case to the statement.
(ii) the if statement following the switch in this preprocessor directive set the variable "package" to a non-existent enum "SO_PACKAGE_DEFAULT" which I believe is a typo for "SO_DEFAULT".

I have also fixed:
(iii) The file ripley_UnitTest.cpp was trying to include the nonexistent file "escript/Esys_MPI.h" instead of "escript/EsysMPI.h", causing the compiler to bounce back an error.

Changed paths

Path Details
Directorytrunk/ripley/src/RipleyDomain.cpp modified , text changed
Directorytrunk/ripley/test/ripley_UnitTest.cpp modified , text changed
Directorytrunk/scons/meerkat_options.py modified , text changed

  ViewVC Help
Powered by ViewVC 1.1.26