Skip to content

add new elements for goolfc/2.6.10#545

Merged
boegel merged 7 commits intoeasybuilders:developfrom
fgeorgatos:contrib_goolfc-2.0.10
Dec 17, 2013
Merged

add new elements for goolfc/2.6.10#545
boegel merged 7 commits intoeasybuilders:developfrom
fgeorgatos:contrib_goolfc-2.0.10

Conversation

@fgeorgatos
Copy link
Copy Markdown
Contributor

Hi,

@ajdecon, @boegel: this is just a re-spin of #477 for an updated goolfc, which is far cleaner to read.
It should be ABI compatible on x86_64 with toolchain goolf/1.6.10. Or, I claim so.

btw. my attempt for a gompic.py is visible in that issue;
somehow, only working variant is class Gompic(Gcc, OpenMPI) and just that; is that normal?

cheerio,
F.

Signed-off-by: Fotis Georgatos <[email protected]>
@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 12, 2013

Jenkins: ok to test

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

drop openmpi-mca-params.conf-1.4.5-local.patch, it is not relevant for most other sites

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.

is this a TODO comment? :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the first part of the patch has to go, indeed;
how about the 2nd part? is that relevant at all? I never got which pax problem it really solves...

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.

the 2nd part may be relevant on some systems, but can also create problems because unknown issues

I've excluded it, at some point we'll have a native Python implementation of untar/unzip, and this will no longer be an issue then I hope

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 16, 2013

a bit of cleanup needed here, i.e. inclusion of a gcccuda easyconfig that combines GCC and CUDA, and a (tested!) GROMACS easyconfig that uses goolfc/1.6.10

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.

We usually align the version numbers of the top and sub toolchains, so bump gompic to 2.0.10 too?

Can you describe what the main motivation was behind 2.0.10? What I want to downgrade GCC 4.8.2 to e.g. 4.6.x? The 0 would not allow that easily, right?

Why the 2.x exactly? Because this features an OpenMPI that has CUDA features enabled? In that case gompic should definitely also be 2.x.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 16, 2013

@fgeorgatos: I'm working on a PR to fix my remarks, hopefully on your side soon.

boegel and others added 5 commits December 16, 2013 21:25
rework goolfc/2.6.10 easyconfigs using gcccuda subtoolchain, add GROMACS easyconfig for goolfc/2.6.10
Signed-off-by: Fotis Georgatos <[email protected]>
@fgeorgatos
Copy link
Copy Markdown
Contributor Author

In the meantime, I found out that hwloc/1.7 series is now classified as (old) (!)
and v1.8 is the stable version, which came out a month earlier. Therefor,
we promoted version and rebuilt everything (save for GCC/4.8.2 & CUDA). Also added,
the new toy from openmpi called netloc/0.5beta which depends upon hwloc & jansson.

This should be ready to test as soon as easybuilders/easybuild-framework#783 gets merged.

Signed-off-by: Fotis Georgatos <[email protected]>
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 17, 2013

building GROMACS/4.6.5-goolfc-2.6.1 worked fine, also testing the netloc build and waiting for Jenkins to give the green light

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 17, 2013

all is well, merging in!

boegel added a commit that referenced this pull request Dec 17, 2013
@boegel boegel merged commit 4131002 into easybuilders:develop Dec 17, 2013
@fgeorgatos fgeorgatos deleted the contrib_goolfc-2.0.10 branch December 17, 2013 11:56
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.

3 participants