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

Superbuild throws error on Ubuntu 18.04 #394

Closed
seandoyle opened this issue Feb 9, 2020 · 11 comments
Closed

Superbuild throws error on Ubuntu 18.04 #394

seandoyle opened this issue Feb 9, 2020 · 11 comments
Labels

Comments

@seandoyle
Copy link

Hi -

I just tried building dcmqi (retrieved via git clone https://github.com/QIICR/dcmqi.git on a new Ubuntu 18.04 machine using the following set of commands:

 mkdir dcmqi-superbuild
 cd dcmqi-superbuild/
 cmake /home/sean/Development/github/dcmqi
 make

which resulted in the error "Dunno about this gcc":

In file included from /home/sean/Development/dcmqi-superbuild/ITK/Modules/ThirdParty/VNL/src/vxl/vcl/vcl_iosfwd.h:8:0,
                 from /home/sean/Development/dcmqi-superbuild/ITK/Modules/ThirdParty/VNL/src/vxl/vcl/vcl_iostream.h:27,
                 from /home/sean/Development/dcmqi-superbuild/ITK/Modules/ThirdParty/VNL/src/vxl/vcl/vcl_deprecated.cxx:4:
/home/sean/Development/dcmqi-superbuild/ITK/Modules/ThirdParty/VNL/src/vxl/vcl/vcl_compiler.h:144:4: error: #error "Dunno about this gcc"

Modules/ThirdParty/VNL/src/vxl/vcl/CMakeFiles/itkvcl.dir/build.make:62: recipe for target 'Modules/ThirdParty/VNL/src/vxl/vcl/CMakeFiles/itkvcl.dir/vcl_deprecated.cxx.o' failed
make[5]: *** [Modules/ThirdParty/VNL/src/vxl/vcl/CMakeFiles/itkvcl.dir/vcl_deprecated.cxx.o] Error 1
CMakeFiles/Makefile2:650: recipe for target 'Modules/ThirdParty/VNL/src/vxl/vcl/CMakeFiles/itkvcl.dir/all' failed
make[4]: *** [Modules/ThirdParty/VNL/src/vxl/vcl/CMakeFiles/itkvcl.dir/all] Error 2
Makefile:151: recipe for target 'all' failed
make[3]: *** [all] Error 2
CMakeFiles/ITK.dir/build.make:112: recipe for target 'ITK-prefix/src/ITK-stamp/ITK-build' failed
make[2]: *** [ITK-prefix/src/ITK-stamp/ITK-build] Error 2
CMakeFiles/Makefile2:260: recipe for target 'CMakeFiles/ITK.dir/all' failed
make[1]: *** [CMakeFiles/ITK.dir/all] Error 2
Makefile:94: recipe for target 'all' failed
make: *** [all] Error 2

I could build the components by scratch - but if there is something I should try first to debug the issue I'm happy to do it. I believe this is is a mismatch of some sort between ITK and newer versions of gcc.

@fedorov
Copy link
Member

fedorov commented Feb 10, 2020

Most likely, both this one and #396 are due to the fact that dcmqi is using a very old, patched, 4.10 version of ITK. I manually stripped off many components which were not possible to disable via cmake to shorten the build time and meet the CI constraints at the time. Probably the best approach would be to upgrade ITK altogether.

Meanwhile, can you use the binary packages: https://github.com/QIICR/dcmqi/releases/tag/latest? dcmqi is still building fine on all platforms by CI.

@fedorov fedorov added the bug label Feb 10, 2020
@seandoyle
Copy link
Author

Yes - not a problem! Thanks.

@pieper
Copy link
Member

pieper commented Feb 10, 2020

I'd actually leave this open - the underlying issue isn't fixed yet.

@pieper pieper reopened this Feb 10, 2020
@fedorov
Copy link
Member

fedorov commented Feb 10, 2020

Yes, definitely - I didn't mean to imply the issue should be closed.

Sean, would binary packages work to address your immediate needs?

@pieper
Copy link
Member

pieper commented Mar 2, 2020

FYI - the dcmqi extension is also broken, and that breaks quantitative reporting:
http://slicer.cdash.org/viewBuildError.php?buildid=1843321

@fedorov
Copy link
Member

fedorov commented Mar 2, 2020

Thanks for the nudge - we fixed dcmqi version to work with the DCMTK pre-updgrade. Once we switch to master, hopefully those errors will go away. I will do this hopefully today once #393 is merged.

@pieper
Copy link
Member

pieper commented Mar 2, 2020

Sound good - I hope there's an easy fix for the extension. The build errors are all about things like CERR and COUT not being defined.

e.g.
D:\D\P\S-0-E-b\DCMQI\libsrc\TID1500Reader.cpp(21): error C2065: 'CERR': undeclared identifier [D:\D\P\S-0-E-b\DCMQI-build\dcmqi-build\libsrc\dcmqi.vcxproj] [D:\D\P\S-0-E-b\DCMQI-build\DCMQI.vcxproj]

@fedorov
Copy link
Member

fedorov commented Mar 2, 2020

Yes, those specific errors have already been fixed in master.

@fedorov
Copy link
Member

fedorov commented Mar 4, 2020

I updated ExtensionIndex, let's see tomorrow.

@fedorov
Copy link
Member

fedorov commented May 19, 2020

I just realized I didn't update the issue, but the Slicer extension does not have build errors.

@jcfr
Copy link
Contributor

jcfr commented Jan 26, 2023

Closing. The following pull-requests likely addressed the issue:

@jcfr jcfr closed this as completed Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants