Skip to content

add gnuplot dependency for OpenFOAM from v2.4.0 to v6#11801

Merged
boegel merged 6 commits intoeasybuilders:developfrom
lexming:20201202233336_new_pr_OpenFOAM6
Feb 12, 2021
Merged

add gnuplot dependency for OpenFOAM from v2.4.0 to v6#11801
boegel merged 6 commits intoeasybuilders:developfrom
lexming:20201202233336_new_pr_OpenFOAM6

Conversation

@lexming
Copy link
Copy Markdown
Contributor

@lexming lexming commented Dec 2, 2020

(created using eb --new-pr)

Follow up on #11770
Needed by easybuilders/easybuild-easyblocks#2256

@lexming lexming changed the title add dependency on gnuplot to OpenFOAM v6 add dependency on gnuplot to OpenFOAM from v2.4.0 to v6 Dec 2, 2020
@lexming lexming added the bug fix label Dec 2, 2020
@lexming lexming added this to the 4.x milestone Dec 2, 2020
@lexming lexming force-pushed the 20201202233336_new_pr_OpenFOAM6 branch from 6d6d643 to f37f57a Compare December 3, 2020 09:22
@easybuilders easybuilders deleted a comment from boegelbot Dec 3, 2020
@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 3, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 9 out of 9 (2 easyconfigs in total)
node376.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/678edd3c9aa0a817f8c65d1558bea634 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 4, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 3 out of 3 (2 easyconfigs in total)
node376.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/0e8bae7c697e748ca3b067c1503046dd for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 6, 2020

@boegelbot please test @ generoso
EB_ARGS=" --include-easyblocks-from-pr 2256 OpenFOAM-6-foss-2019b.eb "
CORE_CNT=16

@boegelbot
Copy link
Copy Markdown
Collaborator

@boegel: Request for testing this PR well received on generoso

PR test command 'EB_PR=11801 EB_ARGS=" --include-easyblocks-from-pr 2256 OpenFOAM-6-foss-2019b.eb " /apps/slurm/default/bin/sbatch --job-name test_PR_11801 --ntasks="16" ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 12231

Test results coming soon (I hope)...

Details

- notification for comment with ID 739556694 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
generoso-x-1 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/6854505d1c27e5b2e0fa7bc6809c5ce3 for a full test report.

@boegel boegel changed the title add dependency on gnuplot to OpenFOAM from v2.4.0 to v6 add gnuplot dependency for OpenFOAM from v2.4.0 to v6 Dec 7, 2020
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 7, 2020

@lexming What's up with the failing test reports?

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 7, 2020

@boegel all easyconfigs in foss/2016b failed because we have a different version of zlib in our toolchain. Re-running...

@boegel boegel modified the milestones: 4.x, 4.4.0 Dec 8, 2020
@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 8, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node355.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/f4420aa9f223523c84a5c060049f88a2 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 8, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node356.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/94d846491607de4d2a0b918091c19570 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 9, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node356.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/09155f42b7906d840a7c85dfa3138ab4 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 9, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
FAILED
Build succeeded for 2 out of 3 (3 easyconfigs in total)
node381.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/d1388163576865e06e56b56ada745a5a for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 9, 2020

OpenFOAM v5 are failing due to a version conflict between Qt4 from ParaView and Qt5 from gnuplot

CMake Error at /theia/home/apps/CO7/skylake/software/CMake/3.10.1-GCCcore-6.4.0/share/cmake-3.10/Modules/FindQt4.cmake:1320 (message):
  Found unsuitable Qt version "5.8.0" from
  /apps/brussel/CO7/skylake/software/Qt5/5.8.0-intel-2017b/bin/qmake, this
  code requires Qt 4.x
Call Stack (most recent call first):
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib64/cmake/paraview-5.4/ParaViewQt.cmake:87 (find_package)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib/cmake/paraview-5.4/Modules/pqApplicationComponents.cmake:11 (pv_find_package_qt)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib/cmake/paraview-5.4/vtkModuleAPI.cmake:45 (include)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib/cmake/paraview-5.4/vtkModuleAPI.cmake:15 (vtk_module_load)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib/cmake/paraview-5.4/vtkModuleAPI.cmake:132 (_vtk_module_config_recurse)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib64/cmake/paraview-5.4/VTKConfig.cmake:135 (vtk_module_config)
  /user/brussel/101/vsc10122/.local/easybuild-skylake/software/ParaView/5.4.1-intel-2017b-mpi/lib64/cmake/paraview-5.4/ParaViewConfig.cmake:51 (include)
  CMakeLists.txt:4 (FIND_PACKAGE)


-- Configuring incomplete, errors occurred!

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 11, 2020

d75a3c7 fixes the aforementioned issue due to mixing of dependencies with Qt4 and Qt5. The solution is to guarantee that dependencies loading Qt4 are last in the list, so that Qt4 has precedende over Qt5. Otherwise the build will fail as described. This problem only happens in the few easyconfigs where ParaView and CGAL depend on Qt4 while gnuplot already uses Qt5.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 14, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
FAILED
Build succeeded for 2 out of 3 (3 easyconfigs in total)
node360.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/acd4fd661f25a7ae45c4f51e1b08c2e4 for a full test report.

@lexming
Copy link
Copy Markdown
Contributor Author

lexming commented Dec 14, 2020

Test report by @lexming
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2256
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
node360.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/e7559bb54a60b547b3ca105bb4ae062a for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 12, 2021

Going in, thanks @lexming!

@boegel boegel merged commit a683955 into easybuilders:develop Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants