Skip to content

add support for Tcl environment modules#728

Merged
stdweird merged 1 commit intoeasybuilders:developfrom
boegel:tcl_modules
Oct 22, 2013
Merged

add support for Tcl environment modules#728
stdweird merged 1 commit intoeasybuilders:developfrom
boegel:tcl_modules

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Oct 22, 2013

First result from the 4th EasyBuild hackathon, required for lots of sites (e.g. JSC, DEISA systems, etc.)

Unit tests pass with this, both for C and Tcl environment modules:

$ export EASYBUILD_MODULES_TOOL=EnvironmentModulesTcl

$ eb PANDAseq-2.5-goolf-1.4.10.eb -ld | grep modulecmd.tcl
== 2013-10-22 17:04:42,929 main.EnvironmentModulesTcl DEBUG Running 'modulecmd.tcl python avail PANDAseq/2.5-goolf-1.4.10 -t' from /Users/kehoste/TMP/tcl...
== 2013-10-22 17:04:42,930 main.EnvironmentModulesTcl DEBUG Running module cmd 'modulecmd.tcl python avail PANDAseq/2.5-goolf-1.4.10 -t'

$ python -m test.framework.suite

....................................................................................................
----------------------------------------------------------------------
Ran 100 tests in 497.025s

OK

stdweird
stdweird reviewed Oct 22, 2013
View reviewed changes
Comment thread easybuild/tools/modules.py
Copy link
Copy Markdown
Contributor

@stdweird stdweird Oct 22, 2013

Choose a reason for hiding this comment

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

append vs insert 0?

Copy link
Copy Markdown
Member Author

@boegel boegel Oct 22, 2013

Choose a reason for hiding this comment

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

modulecmd python --terse avail works (same for lmod), modulecmd.tcl python -t avail doesn't work (no --terse in Tcl env mods, only -t)...

The Tcl environment modules implementation requires (hard) that the --terse is used after the 'subcommand' (avail, list, ...).

stdweird added a commit that referenced this pull request Oct 22, 2013
@stdweird
Merge pull request #728 from boegel/tcl_modules
320c12a 
add support for Tcl environment modules
@stdweird stdweird merged commit 320c12a into easybuilders:develop Oct 22, 2013
@boegel boegel mentioned this pull request Nov 4, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v1.9

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @stdweird