Skip to content

add easyconfig files for ParaStation MPI#759

Merged
boegel merged 12 commits intoeasybuilders:developfrom
carsten-clauss:parastation_configs
May 21, 2014
Merged

add easyconfig files for ParaStation MPI#759
boegel merged 12 commits intoeasybuilders:developfrom
carsten-clauss:parastation_configs

Conversation

@carsten-clauss
Copy link
Copy Markdown
Contributor

Added easyconfig files (configure/make) for ParaStation MPI with download from github (with alternative download paths via the Open ParaStation project on sourceforge / commented out by default).
ParaStation is a robust and efficient cluster middleware, consisting of a high-performance communication layer (ParaStation MPI) which is the MPI library of choice on the JuRoPA systems of the Jülich Supercomputing Centre (JSC). ParaStation MPI is developed and maintained by ParTec Cluster Competence Center GmbH.

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

…eparated config folders for psmpi2 and pscom)
@carsten-clauss
Copy link
Copy Markdown
Contributor Author

Okay, it seems that we need two separated config folders for psmpi2 and pscom in order to get the dependency resolving work. 5e0c6b5 should fix this issue. (psmpi2 is the actual ParaStation MPI while pscom is its low-lever communication layer)

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 2, 2014

Jenkins: ok to test

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

how about using:

    for l in $(prefix)/lib*/lib*.so.3; do \

That would avoid breaking the Makefile on non-openSuSE systems?

Then the patch file can always be applied (it's now apply once, and once not, it should be consistent).

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 2, 2014

@carsten-clauss: thanks for the PR, and welcome to EasyBuild!

I have a couple of (rather minor) remarks, please let us know if you're up for looking into them.

@carsten-clauss
Copy link
Copy Markdown
Contributor Author

@boegel: Thank you for revising the easyconfig files for ParaStation!
I have included most of your remarks (generic patch file, removed superfluous comment and empty line) and have made the sources/source_urls more generic by using the name and version parameters together with a file pattern.
Would this be okay for you?

@fgeorgatos
Copy link
Copy Markdown
Contributor

FAILED
Build succeeded for 0 out of 3
Linux debian 6.0.9, Intel(R) Xeon(R) CPU L5640 @ 2.27GHz, Python 2.6.6
See https://gist.github.com/c57fde2d974d1e73dad5 for a full test report.

@fgeorgatos
Copy link
Copy Markdown
Contributor

bother not with the above report, I was just trying the option --from-pr on a random system

@boegel
Copy link
Copy Markdown
Member

boegel commented May 16, 2014

FAILED
Build succeeded for 0 out of 3
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/c227328fba73ca282b5f for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 16, 2014

@carsten-clauss: build fails on my end, see test report, more specifically the following error in the build log: https://gist.github.com/boegel/034e5f649baeb6abaeda#file-pscom-5-0-43-gcc-4-8-2_partial-log-L486

/usr/bin/ld: cannot find -lpopt

Does that ring a bell for you? Missing library?

add missing popt dependency for pscomp, fix psmpi2 patch file, style fixes
@boegel
Copy link
Copy Markdown
Member

boegel commented May 19, 2014

@carsten-clauss: I'm still running into the issue below... Are you using a tarball as it was downloaded via https://github.com/ParaStation/psmpi/archive? Because it seems like the build procedure requires it's an actual git clone (not an exported version of the git tree). Any way around that?

== FAILED: Installation ended unsuccessfully (build directory:
... cmd " make install " exited with exitcode 2 and output:
No .git or .svn found
make -C mpich2 install

@carsten-clauss
Copy link
Copy Markdown
Contributor Author

@boegel: Thank you for reviewing and pointing out this issue!
In fact, one of the install scripts tries to figure out the current version of psmpi2 by checking for local git or svn information -- of course without success.
Therefore, I have added a further patch that should fix this issue in the related script.

boegel and others added 2 commits May 20, 2014 14:06
resolve version issue via creating of a VERSION file through premakeopts, enforce non-parallel build
@boegel
Copy link
Copy Markdown
Member

boegel commented May 20, 2014

Test report by @boegel
FAILED
Build succeeded for 2 out of 4
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/3d115409a91cbda1f793 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 20, 2014

@carsten-clauss : ignore the build failures in the test report, the patch file is not being applied correctly because of a bug in the EasyBuild framework (see also #876 (comment))

I'll fix it and retest, this should be ready to go in now.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 20, 2014

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/026d3615f088a82cf3d2 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 20, 2014

depends on easybuilders/easybuild-framework#933

@boegel
Copy link
Copy Markdown
Member

boegel commented May 21, 2014

Test report by @boegel
SUCCESS
Build succeeded for 4 out of 4
Linux SL 6.5, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/1f47e1eee6de13a60013 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 21, 2014

bug fix for git diff patch files has been merged, so this is ready to go in now (in time for EasyBuild v1.13)

Thanks @carsten-clauss!

boegel added a commit that referenced this pull request May 21, 2014
add easyconfig files for ParaStation MPI
@boegel boegel merged commit 3da35e1 into easybuilders:develop May 21, 2014
@carsten-clauss
Copy link
Copy Markdown
Contributor Author

@boegel: Thank you very much!

@carsten-clauss carsten-clauss deleted the parastation_configs branch May 21, 2014 15:13
@fgeorgatos
Copy link
Copy Markdown
Contributor

Test report by @fgeorgatos
SUCCESS
Build succeeded for 4 out of 4
Linux debian 6.0.9, Intel(R) Xeon(R) CPU E5-2660 0 @ 2.20GHz, Python 2.6.6
See https://gist.github.com/b2bd10b23309229330f9 for a full test report.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants