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

MODULEPATH definitions under /etc/profile.d must be w. guarded variable #2

Closed
fgeorgatos opened this issue Feb 15, 2016 · 5 comments
Closed
Assignees

Comments

@fgeorgatos
Copy link
Collaborator

"""
Verify that MODULEPATH definitions under /etc/profile.d are always w. guarded variable for either Lmod/Tmod ## reported bug #10582, handled on 13/11/2015
"""

I believe this was fixed as of Bright v.7.0-44; is it the case that all of 7.x variants have the bugfix?

Of special interest is the Bright/7.0 against Centos6 case, see #9

@fgeorgatos
Copy link
Collaborator Author

as of 31/3/2016:

  • Tmod is handled with 7.0-92 patches and onwards, since some guarding logic has been added
  • Lmod/6.0, as it comes from EPEL repo, is not proper because it has hardwired MODULEPATH :-(

@fgeorgatos
Copy link
Collaborator Author

Wow, one extra group of people introducing the exact same bug we started with:
https://github.com/openhpc/ohpc/blob/issue_178/components/admin/lmod/SPECS/lmod.spec

a hardwired $MODULEPATH breaks module use across inherited shells, please avoid that!

@boegel
Copy link
Collaborator

boegel commented Apr 15, 2016

@fgeorgatos is that still the case with recent Lmod versions? I think these pick up an existing $MODULEPATH without too much trouble?

Can you explain what problems you run into exactly, and share error messages, etc?

@fgeorgatos
Copy link
Collaborator Author

fgeorgatos commented May 4, 2016

@boegel: sry, I only now see your message; as discussed over easybuild update calls, the following was broken across several - really many- environment modules implementations (mostly at packaging level providing for /etc/profile.d scripts):

echo $MODULEPATH
mkdir $HOME/privatemodules
module use $HOME/privatemodules
echo $MODULEPATH
bash
echo $MODULEPATH

As of commit 1e9ba21 in this branch, this issue should no longer exist for Lmod, nor for Tmod as it comes from Bright (non-blocking improvements expected on either .spec file, though)

Fedora picked up on the bug last week and got it fixed for Lmod, too:
https://bugzilla.redhat.com/show_bug.cgi?id=1326075
https://bodhi.fedoraproject.org/updates/Lmod-6.3.1-3.el6
I highly recommend to look at that as a reference implementation for many others (plz verify this)

@fgeorgatos
Copy link
Collaborator Author

this is guarded now sufficiently, as regards generic Lmod/Tmod deployments, so resolving
(a little internal issue #12041 is considered related but not a blocker)

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

No branches or pull requests

3 participants