Skip to content

Conversation

@N-Dekker
Copy link
Member

Moved the #include "itkOpenCLContext.h" from elxElastixMain.h to elxElastixMain.cxx and elxTransformixMain.cxx, hoping to ease fixing the error:

/work/_skbuild/linux-x86_64-3.5/cmake-build/_deps/elx-src/Core/Kernel/elxElastixMain.h:33:10: fatal error: 'itkOpenCLContext.h' file not found

at https://github.com/InsightSoftwareConsortium/ITKElastix/pull/58/checks?check_run_id=1096122268#step:4:2894

(Pull request InsightSoftwareConsortium/ITKElastix#58 "ENH: Switch to latest version of SuperElastix/elastix (using ITK 5.1.1)")

Moved the `#include "itkOpenCLContext.h"` from elxElastixMain.h to elxElastixMain.cxx and elxTransformixMain.cxx, hoping to ease fixing the error:

> /work/_skbuild/linux-x86_64-3.5/cmake-build/_deps/elx-src/Core/Kernel/elxElastixMain.h:33:10: fatal error: 'itkOpenCLContext.h' file not found

at https://github.com/InsightSoftwareConsortium/ITKElastix/pull/58/checks?check_run_id=1096122268#step:4:2894

(Pull request InsightSoftwareConsortium/ITKElastix#58 "ENH: Switch to latest version of SuperElastix/elastix (using ITK 5.1.1)")
@N-Dekker N-Dekker changed the title COMP: Move #include "itkOpenCLContext.h" from elxElastixMain.h to "itkOpenCLContext.h" COMP: Move #include "itkOpenCLContext.h" from elxElastixMain.h to elxElastixMain.cxx and elxTransformixMain.cxx Sep 11, 2020
@N-Dekker
Copy link
Member Author

@dpshamonin Denis, for your information, it seems nicer to move the #include "itkOpenCLContext.h" from elxElastixMain.h to those two CXX files.

@N-Dekker N-Dekker marked this pull request as ready for review September 11, 2020 13:11
@N-Dekker N-Dekker merged commit beda57f into develop Sep 11, 2020
@N-Dekker N-Dekker deleted the Move-include-itkOpenCLContext.h branch September 11, 2020 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants