On 20/02/2017 14:53, Siddiqui, Shahzeb wrote:
Do you want me to update the PR with the name IntelClusterChecker.
I think that makes sense, yes.
PATH variable needs to be set to use CLCK.
OK, but let's follow this up in your pull request
https://github.com/hpcugent/easybuild-easyconfigs/pull/4156/ ?
regards,
Kenneth
-----Original Message-----
From: [email protected]
[mailto:[email protected]] On Behalf Of Kenneth Hoste
Sent: Monday, February 20, 2017 5:05 AM
To: [email protected]
Subject: Re: [easybuild] Module File for Intel clck
On 20/02/2017 03:11, Fotis Georgatos wrote:
On Feb 17, 2017, at 7:53 PM, Kenneth Hoste <[email protected]> wrote:
It's unclear to me what you consider to be wrong in this module file exactly?
Is something missing, or are some environment variables set incorrectly?
I believe you would expect some *PATH* variables to be defined there, no?!
See also
https://github.com/hpcugent/easybuild-easyconfigs/pull/4156/files, where $PATH
& co are being specified (although perhaps incorrectly?).
Since we’re still on this topic: clck is not a spectacular name in
terms of user friendliness; perhaps adding and intel- or even i prefix may make
it more palatable to first time cluster testers...
+1, maybe should be IntelClusterChecker?
K.
