Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Sep 28, 2018

Strictly speaking, there's nothing wrong with sticking to a .modulerc in Tcl syntax, but it has the indirect implication that tcslh must be available , causing problems like (reported by @vanzod):

$ module use /tmp/eb-93S43S/tmpR_HaVA/all/Core
sh: 1: /usr/bin/tclsh: not found
Lmod has detected the following error:  Unable to parse: "/tmp/eb-93S43S/tmpR_HaVA/all/Core/Java/.modulerc". Aborting!

These changes spit out the .modulerc in Lua syntax when Lua is used as module syntax and if the Lmod version is recent enough (7.8 or newer).

The ModuleRC easyblock that was added in easybuilders/easybuild-easyblocks#1503 will need to made aware of this too, since when Lua syntax is used for the .modulerc file the filename should be .modulerc.lua...

@boegel boegel added this to the 3.7.1 milestone Sep 28, 2018
@boegel boegel requested a review from vanzod September 28, 2018 13:59
@easybuilders easybuilders deleted a comment from boegelbot Sep 28, 2018
@easybuilders easybuilders deleted a comment from boegelbot Sep 28, 2018
@vanzod
Copy link
Member

vanzod commented Sep 28, 2018

I have been testing this together with easybuilders/easybuild-easyblocks#1533 and they definitely solve the tclsh problem by creating a .modulerc.lua. However I am still having troubles installing Java/1.8, which returns a module not found error. I cannot understand if it is in any way related.
easybuild-Java-1.8-20180928.172250.uEVRe.log

@zao
Copy link
Contributor

zao commented Sep 29, 2018

@vanzod I'm experiencing a similar Java problem with 3.7.0 on 6.6 series of Lmod, but not with the current 7 version - easybuilders/easybuild#462

@boegel
Copy link
Member Author

boegel commented Sep 29, 2018

Based on the log, @vanzod is seeing the problem with Lmod 7.8.6, but it may be a different problem with the new .modulerc.lua...

@boegel
Copy link
Member Author

boegel commented Sep 29, 2018

@vanzod You're hitting the same problem as @guest1604 is reporting in easybuilders/easybuild-easyblocks#1534, based on this line from the log:

== 2018-09-28 17:22:50,885 modulerc.py:78 INFO Adding module version alias for Core/Java/1.8.0_181 to /tmp/eb-Yr5Ae5/tmpmDy216/all/Core/Java/.modulerc.lua

@migueldiascosta
Copy link
Member

@vanzod can you check again now that easybuilders/easybuild-easyblocks#1535 was merged?

@vanzod
Copy link
Member

vanzod commented Oct 1, 2018

@migueldiascosta Test passed. Now it works like a charm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants