Skip to content

{numlib}[intel/2017b] CGAL v4.11 w/ Python 3.6.3#5539

Merged
boegel merged 2 commits intoeasybuilders:developfrom
Helios07:20171218144448_new_pr_CGAL411
Jan 23, 2018
Merged

{numlib}[intel/2017b] CGAL v4.11 w/ Python 3.6.3#5539
boegel merged 2 commits intoeasybuilders:developfrom
Helios07:20171218144448_new_pr_CGAL411

Conversation

@Helios07
Copy link
Copy Markdown

@Helios07 Helios07 commented Dec 18, 2017 •
edited by boegel
Loading

(created using eb --new-pr)
Depends on #5537

@boegel boegel added the update label Dec 18, 2017
@boegel boegel added this to the 3.5.1 milestone Dec 18, 2017
@easybuilders easybuilders deleted a comment from boegelbot Dec 18, 2017
boegel
boegel reviewed Dec 19, 2017
View reviewed changes
easybuild/easyconfigs/c/CGAL/CGAL-4.11-intel-2017b-Python-3.6.3.eb Outdated
('MPFR', '3.1.5'),
('GMP', '6.1.2'),
('libGLU', '9.0.0'),
('Qt5', '5.8.0'),
Copy link
Copy Markdown
Member

@boegel boegel Dec 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Helios07 Existing CGAL 4.11 easyconfigs use Qt 4.x rather than Qt5, can you explain the jump to Qt5, do you have a specific reason for it?

In addition, there's a Qt5 5.10.0 (and 5.9.x) already, why not go with the most recent Qt5?

Copy link
Copy Markdown
Author

@Helios07 Helios07 Dec 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@boegel Qt 5.9 was not building for my setup yet, so I kept using 5.8. I am not sure about the differences between Qt 4 and 5 so I tried to use Qt 5. Would you suggest to stick with version 4 here?

Copy link
Copy Markdown
Member

@boegel boegel Dec 27, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably makes sense to go forward with Qt5, but then using a more recent version would make sense.

In #5344, the CGAL easyconfig uses Qt5 5.9.2 (but there's a 5.9.3 already too).

Copy link
Copy Markdown
Author

@Helios07 Helios07 Jan 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qt 5.9 won't build on my CentOS 7 system with EasyBuild. I am searching for a solution.

Copy link
Copy Markdown
Member

@boegel boegel Jan 11, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Helios07 Does easybuilders/easybuild-easyblocks#1328 help?

Copy link
Copy Markdown
Author

@Helios07 Helios07 Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@boegel I still cannot build Qt 5.9.3 with the Intel toolchain and the error messages are not helpful. It worked with foss, but I cannot find, what I am doing wrong.

Copy link
Copy Markdown
Member

@boegel boegel Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you using EasyBuild v3.5.1? Can you share the build log with the errors somewhere, e.g. via gist.github.com?

Copy link
Copy Markdown
Author

@Helios07 Helios07 Jan 19, 2018 •
edited by boegel
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I use EasyBuild 3.5.1. You can fine my build log here:
https://uni-muenster.sciebo.de/s/jNcjLmsNizqJw9g

The output of the build was:
[h_zimm01@palma003 ~]$ eb Qt5-5.9.3-intel-2017b.eb
== temporary log file in case of crash /tmp/eb-3lw6m3/easybuild-d2LMFx.log
== processing EasyBuild easyconfig /home/h/h_zimm01/Qt5-5.9.3-intel-2017b.eb
== building and installing MPI/intel/2017.4.196-GCC-6.4.0-2.28/impi/2017.3.196/Qt5/5.9.3...
== fetching files...
== creating build dir, resetting environment...
== unpacking...
== patching...
== preparing...
== configuring...
== building...
== FAILED: Installation ended unsuccessfully (build directory: /tmp/easybuild/build/Qt5/5.9.3/intel-2017b): build failed (first 300 chars): cmd " LD_LIBRARY_PATH=/tmp/easybuild/build/Qt5/5.9.3/intel-2017b/qt-everywhere-opensource-src-5.9.3/qtbase/lib:/tmp/easybuild/build/Qt5/5.9.3/intel-2017b/qt-everywhere-opensource-src-5.9.3/qtdeclarative/lib:$LD_LIBRARY_PATH  make -j 32 " exited with exit code 2 and output:
cd qtbase/ && ( test -e Ma
== Results of the build can be found in the log file(s) /tmp/eb-3lw6m3/easybuild-Qt5-5.9.3-20180119.142624.YhlqZ.log
ERROR: Build of /home/h/h_zimm01/Qt5-5.9.3-intel-2017b.eb failed (err: 'build failed (first 300 chars): cmd " LD_LIBRARY_PATH=/tmp/easybuild/build/Qt5/5.9.3/intel-2017b/qt-everywhere-opensource-src-5.9.3/qtbase/lib:/tmp/easybuild/build/Qt5/5.9.3/intel-2017b/qt-everywhere-opensource-src-5.9.3/qtdeclarative/lib:$LD_LIBRARY_PATH  make -j 32 " exited with exit code 2 and output:\ncd qtbase/ && ( test -e Ma')

Copy link
Copy Markdown
Member

@boegel boegel Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eek, that's a nasty error...

In file included from deps/geojsonvt/6.3.0/include/mapbox/geojsonvt/simplify.hpp(3),
                 from deps/geojsonvt/6.3.0/include/mapbox/geojsonvt/convert.hpp(3),
                 from deps/geojsonvt/6.3.0/include/mapbox/geojsonvt.hpp(3),
                 from src/mbgl/annotation/shape_annotation_impl.hpp(4),
                 from src/mbgl/annotation/fill_annotation_impl.hpp(3),
                 from src/mbgl/annotation/fill_annotation_impl.cpp(1):
deps/geojsonvt/6.3.0/include/mapbox/geojsonvt/types.hpp(63): error: cannot determine the exception specification of the default constructor due to a circular dependency
      using container_type::container_type;

No idea what's going on here, but it looks like this may be a compiler bug.

This, on top of the problem with using Qt5 as indirect dependency we're running into in #5344 make me reconsider the switch to Qt5, I think we should stick with the latest Qt 4.x instead?

I've just opened a PR for Qt 4.8.7 with intel/2017b at #5682, can you try changing the Qt in this CGAL easyconfig to 4.8.7, and using that instead?

Copy link
Copy Markdown
Author

@Helios07 Helios07 Jan 23, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem. It also builds with your Qt 4.8.7 package.

@boegel boegel modified the milestones: 3.5.1, 3.6.0 Jan 11, 2018
@boegel boegel changed the title {numlib}[intel/2017b] CGAL v4.11 {numlib}[intel/2017b] CGAL v4.11 w/ Python 3.6.3 Jan 19, 2018
@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 23, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2539.golett.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/da8d1a4513682c7390e4b9c9b3e6035c for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 23, 2018

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2016.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/8fade616b1f7880cfa40758527b1e5f7 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jan 23, 2018

Going in, thanks @Helios07!

@boegel boegel merged commit 6463131 into easybuilders:develop Jan 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

update

Projects

None yet

Milestone

3.5.2

Development

Successfully merging this pull request may close these issues.

2 participants

@Helios07 @boegel