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

Build fails with duplicate symbols when using external ITK in superbuild #82

Open
marcelluethi opened this issue Aug 27, 2014 · 2 comments
Open

Build fails with duplicate symbols when using external ITK in superbuild #82

marcelluethi opened this issue Aug 27, 2014 · 2 comments
Assignees
@marcelluethi

Comments

@marcelluethi
Copy link
Member

The problem is related to HDF5, which is included once from ITK and once from statismo.
@marcelluethi marcelluethi self-assigned this Aug 27, 2014
@marcelluethi marcelluethi mentioned this issue Oct 3, 2014
Open
@itwasabhi
Copy link

Has there been any update on this issue? I'm having the same problem.

marcelluethi added a commit that referenced this issue Sep 1, 2015
@marcelluethi
Change of include order to fix #82
5357bda 
If statismo is used with an external ITK, the order of the headers need to be such that itk's itk_hdf5.h is included
first. If not, duplicate imports and/or duplicate symbols during linking can occur.
@marcelluethi
Copy link
Member Author

The issue is related to the order the headers are included. One needs to make sure, that the header itk_hdf5.h is always included first. This can be achieved by including the file statismoITKConfig.h as a first header file.

I just pushed a commit 5357bda, which should solve that problem. Can you give it a try and let me know if it works for you?

In a final solution, we should have a compile time error if the include order is not correct. Unfortunately I currently don't know how to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marcelluethi marcelluethi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@marcelluethi @itwasabhi