{cae}[foss/2017b] OpenFOAM 4.1 (REVIEW) by renatopancheri · Pull Request #5344 · easybuilders/easybuild-easyconfigs

renatopancheri · 2017-11-15T15:52:36Z

(created using eb --new-pr)

boegel · 2017-12-27T12:49:39Z

easybuild/easyconfigs/c/CGAL/CGAL-4.11-foss-2017b-Python-2.7.14.eb

+configopts += "-DBLAS_INCLUDE_DIR=$EBROOTOPENBLAS/include "
+configopts += "-DBLAS_LIBRARIES=$EBROOTOPENBLAS/lib/libopenblas.%s " % SHLIB_EXT
+configopts += "-DLAPACK_INCLUDE_DIR=$EBROOTSCALAPACK/include "
+configopts += "-DLAPACK_LIBRARIES=$EBROOTSCALAPACK/lib/libscalapack.a "


There's a lot of stuff here that could/should be handled by the CGAL easyblock instead...

boegel · 2017-12-27T12:50:02Z

easybuild/easyconfigs/c/CGAL/CGAL-4.11-foss-2017b-Python-2.7.14.eb

+configopts += "-DBLAS_LIBRARIES=$EBROOTOPENBLAS/lib/libopenblas.%s " % SHLIB_EXT
+configopts += "-DLAPACK_INCLUDE_DIR=$EBROOTSCALAPACK/include "
+configopts += "-DLAPACK_LIBRARIES=$EBROOTSCALAPACK/lib/libscalapack.a "
+moduleclass = 'numlib'


style remark: missing empty line above

boegel · 2017-12-27T12:52:09Z

easybuild/easyconfigs/p/ParaView/ParaView-5.2.0-foss-2017b-mpi.eb

+    ('X11', '20171023'),
+    ('Mesa', '17.2.5'),
+    ('libGLU', '9.0.0'),
+    ('Qt', '4.8.7'),


@renatopancheri Can/should we also use Qt5 here?

boegel · 2017-12-27T12:54:43Z

easybuild/easyconfigs/p/ParaView/ParaView-5.2.0-foss-2017b-mpi.eb

+configopts += '-DVTK_WRAP_PYTHON=ON -DPARAVIEW_ENABLE_MATPLOTLIB=ON '
+configopts += '-DPARAVIEW_QT_VERSION=4 -DQT_QMAKE_EXECUTABLE=$EBROOTQT/bin/qmake '
+# if you want to build server only Paraview, uncomment the following line:
+configopts += '-DVTK_USE_X=OFF '


@renatopancheri Existing ParaView easyconfigs have this commented out, could you do that here too?

boegel · 2018-01-11T17:22:50Z

Test report by @boegel
FAILED
Build succeeded for 6 out of 8 (8 easyconfigs in this PR)
node2428.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/6d50fe23691915417265d07f6636ea67 for a full test report.

boegel · 2018-01-11T19:04:22Z

Test report by @boegel
FAILED
Build succeeded for 6 out of 8 (8 easyconfigs in this PR)
node2133.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/57ebdc2a7d3749eb2e9eef5a542367d0 for a full test report.

boegel · 2018-01-18T12:03:40Z

Test report by @boegel
FAILED
Build succeeded for 7 out of 8 (8 easyconfigs in this PR)
node2061.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/062538b0c7b737f116992093a0e592c3 for a full test report.

boegel · 2018-01-18T12:24:38Z

Test report by @boegel
FAILED
Build succeeded for 7 out of 8 (8 easyconfigs in this PR)
node2570.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/e52cc8764a157da2f524600d36f7e2eb for a full test report.

boegel · 2018-01-19T12:53:49Z

This seems to be the actual issue in the failing test report:

CMake Error at CMakeLists.txt:45 (QT4_WRAP_CPP):
  Unknown CMake command "QT4_WRAP_CPP".


-- Configuring incomplete, errors occurred!
See also "/xxx/software/OpenFOAM/4.1-foss-2017b/OpenFOAM-4.1/applications/utilities/postProcessing/graphics/PVReaders/PVblockMeshReader/PVblockMeshReader/Make/linux64GccDPInt32Opt/CMakeFiles/CMakeOutput.log".
make[2]: *** [PVReaders] Error 1
make[1]: *** [graphics] Error 2

This tells me that OpenFOAM 4.1 isn't happy with the use of Qt5, and it requires Qt 4.x instead?

Did this actually work for you @renatopancheri?

boegel · 2018-01-19T12:54:44Z

Some more info:

 $ grep -R QT4_WRAP_CPP /xxx/software/OpenFOAM/4.1-foss-2017b/
/xxx/software/OpenFOAM/4.1-foss-2017b/OpenFOAM-4.1/applications/utilities/postProcessing/graphics/PVReaders/PVFoamReader/PVFoamReader/CMakeLists.txt:QT4_WRAP_CPP(MOC_SRCS pqPVFoamReaderPanel.h)
/xxx/software/OpenFOAM/4.1-foss-2017b/OpenFOAM-4.1/applications/utilities/postProcessing/graphics/PVReaders/PVblockMeshReader/PVblockMeshReader/CMakeLists.txt:QT4_WRAP_CPP(MOC_SRCS pqPVblockMeshReaderPanel.h)
/xxx/software/OpenFOAM/4.1-foss-2017b/OpenFOAM-4.1/applications/utilities/postProcessing/graphics/PV3Readers/PV3FoamReader/PV3FoamReader/CMakeLists.txt:QT4_WRAP_CPP(MOC_SRCS pqPV3FoamReaderPanel.h)
/xxx/software/OpenFOAM/4.1-foss-2017b/OpenFOAM-4.1/applications/utilities/postProcessing/graphics/PV3Readers/PV3blockMeshReader/PV3blockMeshReader/CMakeLists.txt:QT4_WRAP_CPP(MOC_SRCS pqPV3blockMeshReaderPanel.h)

boegel · 2018-01-19T17:00:50Z

easybuild/easyconfigs/c/CGAL/CGAL-4.11-foss-2017b-Python-2.7.14.eb

+builddependencies = [
+    ('CMake', '3.9.5'),
+    ('Eigen', '3.3.4', '', True),
+    ('Qt5', '5.9.2'),


@renatopancheri You may need to move this back to Qt 4.8.7 too?

renatopancheri · 2018-01-19T17:39:45Z

with my machine it builds using Qt4 and not with Qt5

/applications/utilities/postProcessing/graphics/PVReaders/PVblockMeshReader/PVblockMeshReader/CMakeLists.txt:45:

QT4_WRAP_CPP`

-https://openfoam.org/download/source/software-for-compilation/

Note: If Qt 5 is also installed in Ubuntu, the following should be executed before running makeParaView:
export QT_SELECT=qt4

those 2 reasons made me go back to Qt4

migueldiascosta · 2019-02-13T04:55:52Z

@renatopancheri is this still relevant?

migueldiascosta · 2019-05-08T03:31:03Z

@renatopancheri, this PR is being closed for the following reason(s): no activity for > 6 months, obsoleted by more recent PRs.
Please don't hesitate to reopen this PR or add a comment if you feel this contribution is still relevant.
For more information on our policy w.r.t. closing PRs, see https://easybuild.readthedocs.io/en/latest/Contributing.html#why-a-pull-request-may-be-closed-by-a-maintainer

renatopancheri and others added 30 commits November 15, 2017 17:52

adding easyconfigs: OpenFOAM-4.1-foss-2017b.eb

5ae1f5c

whitespaces + using existing patch for intltool-perl

404f7a0

whitespaces + using existing patch for intltool-perl

bd8b08c

whitespaces + using existing patch for intltool-perl

0a96a90

whitespaces + using existing patch for intltool-perl

df498ea

whitespaces + using existing patch for intltool-perl

277afc1

whitespaces + using existing patch for intltool-perl

77d243b

whitespaces + using existing patch for intltool-perl

69f465c

whitespaces + using existing patch for intltool-perl

55e56b0

whitespaces + using existing patch for intltool-perl

6ddb7e1

whitespaces + using existing patch for intltool-perl

2712174

whitespaces + using existing patch for intltool-perl

609b5e6

whitespaces + using existing patch for intltool-perl

029e68d

whitespaces + using existing patch for intltool-perl

13cd607

whitespaces + using existing patch for intltool-perl

5b0329b

whitespaces + using existing patch for intltool-perl

3b7e524

whitespaces + using existing patch for intltool-perl

a3364f7

whitespaces + using existing patch for intltool-perl

36b4dda

whitespaces + using existing patch for intltool-perl

a7a0a0a

whitespaces + using existing patch for intltool-perl

4fbee16

whitespaces + using existing patch for intltool-perl

f5448e7

whitespaces + using existing patch for intltool-perl

b8704ab

whitespaces + using existing patch for intltool-perl

1e84790

whitespaces + using existing patch for intltool-perl

706a6cd

whitespaces + using existing patch for intltool-perl

13f2b66

whitespaces + using existing patch for intltool-perl

b6e3c94

whitespaces + using existing patch for intltool-perl

d68ade7

adding Paraview Patch file

2e98dcb

intltool patch perl

fc2b09c

dropping bzip2 foss2017 because conflicts

08214fc

renatopancheri added 2 commits December 21, 2017 16:58

Delete Mesa-17.2.4-foss-2017b.eb

364d04e

Update ParaView-5.2.0-foss-2017b-mpi.eb

7e3ac87

easybuilders deleted a comment from boegelbot Dec 27, 2017

boegel mentioned this pull request Dec 27, 2017

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

Merged

boegel requested changes Dec 27, 2017

View reviewed changes

renatopancheri added 2 commits January 2, 2018 10:48

removed stuff from CGAL + qt5 ParaView

81c341a

correcting style

ced16ad

corrected checksum

bd388b2

boegel changed the title ~~New EasyConfigs for OpenFOAM with the foss-2017b toolchain~~ {cae}[foss/2017b] OpenFOAM 4.1 (REVIEW) Jan 19, 2018

renatopancheri and others added 3 commits January 19, 2018 17:53

returning to Qt4

aea7b6a

Delete Qt-4.8.7-foss-2017b.eb

f9c06ba

Delete SCOTCH-6.0.4-foss-2017b.eb

aa27b4e

boegel requested changes Jan 19, 2018

View reviewed changes

Delete CGAL-4.11-foss-2017b-Python-2.7.14.eb

5b02dac

migueldiascosta closed this May 8, 2019

migueldiascosta added this to the 3.x milestone May 8, 2019

boegel added the merge sprint label May 9, 2019

Conversation

renatopancheri commented Nov 15, 2017

Uh oh!

boegel Dec 27, 2017

Choose a reason for hiding this comment

Uh oh!

boegel Dec 27, 2017

Choose a reason for hiding this comment

Uh oh!

boegel Dec 27, 2017

Choose a reason for hiding this comment

Uh oh!

boegel Dec 27, 2017

Choose a reason for hiding this comment

Uh oh!

boegel commented Jan 11, 2018

Uh oh!

boegel commented Jan 11, 2018

Uh oh!

boegel commented Jan 18, 2018

Uh oh!

boegel commented Jan 18, 2018

Uh oh!

boegel commented Jan 19, 2018

Uh oh!

boegel commented Jan 19, 2018

Uh oh!

boegel Jan 19, 2018

Choose a reason for hiding this comment

Uh oh!

renatopancheri commented Jan 19, 2018

Uh oh!

migueldiascosta commented Feb 13, 2019

Uh oh!

migueldiascosta commented May 8, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants