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.

Sign up for GitHub

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

Jump to bottom

Project should be defined in the top-level CMakeLists.txt #8

Closed
kaspermarstal opened this issue May 23, 2017 · 1 comment
Closed

Project should be defined in the top-level CMakeLists.txt #8

kaspermarstal opened this issue May 23, 2017 · 1 comment
Assignees
@N-Dekker
Labels
bug

Comments

@kaspermarstal
Copy link
Member

No description provided.

@mstaring mstaring assigned mstaring and N-Dekker and unassigned mstaring Jun 1, 2017
@mstaring mstaring added the bug label Jun 1, 2017
@schmiedc schmiedc mentioned this issue Jun 20, 2017
Closed
@N-Dekker
Copy link
Member

N-Dekker commented Jun 20, 2017 •
edited
Loading

If I remember correctly, we agreed to move the five subdirectories of "elastix/src/" to the root ("elastix/"): Common, Components, Core, and Testing. And then just remove the "src" directory. Please correct me if I'm wrong!

N-Dekker added a commit that referenced this issue Jun 22, 2017
@N-Dekker
BUG: Fixed issue #8 (Project should be defined in the top-level CMake…
0a87c9d 
…Lists.txt):

1. Removed CMakeLists, as it will be replaced by (an adapted version of) src/CMakeLists.
2. Moved the five subdirectories and the CMakeLists from "src/" to the root, preparing to fixing issue #8.
3. Moved 'project' command upward, and copied 'cmake_minimum_required' and 'include' command from https://github.com/SuperElastix/elastix/blob/develop/CMakeLists.txt
4. Fixed paths, now that src subdirs have been moved up to the root.
5. Removed "src" from comment.
N-Dekker added a commit that referenced this issue Jun 22, 2017
@N-Dekker
BUG: Merged issue #8 (Project should be defined in the top-level CMak…
5b554d4 
…eLists.txt)
N-Dekker added a commit to SuperElastix/SuperElastix that referenced this issue Jun 26, 2017
@N-Dekker
ENH: Added #47 (IncredBuild support) to elastix (within SuperElastix-…
590bf89 
…SuperBuild), which was enabled by SuperElastix/elastix#8 (Project should be defined in the top-level CMakeLists.txt)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@N-Dekker N-Dekker

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kaspermarstal @mstaring @N-Dekker