Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suppress creation of module symlinks for hierarchical MNS #1173

Merged
merged 1 commit into from Feb 12, 2015

Conversation

geimer
Copy link
Contributor

@geimer geimer commented Feb 12, 2015

In the context of a hierarchical module naming scheme, the default symlink farm prefixing the module path with the moduleclass easyconfig parameter is of very limited use. First, it creates "strange" modulefile layouts such as

numlib/
    Compiler/GCC/4.9.2/
        OpenBLAS
    MPI/GCC/4.9.2/OpenMPI/1.8.4/
        FFTW
        ScaLAPACK

which are hard to use in a sensible way. And second, the compiler and MPI modules only extend $MODULEPATH for the regular hierarchy, resulting in weird effects when loading these modules. Therefore, it's probably best to suppress the generation of these symlinks entirely.

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Member

boegel commented Feb 12, 2015

@geimer: makes perfect sense, and if we merge this in before EB v2.0 we have a good excuse to break this ;)

@boegel
Copy link
Member

boegel commented Feb 12, 2015

Jenkins: ok to test

@hpcugentbot
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/1330/
Test PASSed.

@boegel
Copy link
Member

boegel commented Feb 12, 2015

Thanks @geimer, going in!

boegel added a commit that referenced this pull request Feb 12, 2015
Suppress creation of module symlinks for hierarchical MNS
@boegel boegel merged commit 5eb3377 into easybuilders:develop Feb 12, 2015
@geimer geimer deleted the hierarchical_mns branch February 12, 2015 17:06
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.

None yet

3 participants