Skip to content

enhance bootstrap script to also support Lmod and modulecmd.tcl#869

Merged
boegel merged 5 commits intoeasybuilders:developfrom
boegel:enhance_bootstrap
Feb 26, 2014
Merged

enhance bootstrap script to also support Lmod and modulecmd.tcl#869
boegel merged 5 commits intoeasybuilders:developfrom
boegel:enhance_bootstrap

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 24, 2014

No description provided.

@boegel boegel added this to the v1.12 milestone Feb 24, 2014
Comment thread easybuild/scripts/bootstrap_eb.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also print this at the end with the things to do/variables to set.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@boegel Shouldn't the bootstrap script add $HOME/.local/easybuild/modules/all to $MODULEPATH? I've seen some users at the JSC that didn't know that they had to extend $MODULEPATH for eb to work it's magic. Or maybe just emit a statement at the end of the bootstrap that the user should adjust $MODULEPATH accordingly?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@stdweird: OK, will do.
@pforai: That's already printed, see:

[[INFO]] Done!
[[INFO]] EasyBuild v1.11.0 was installed to /tmp/ebbootstraptest, so make sure your $MODULEPATH includes /tmp/ebbootstraptest/modules/all
[[INFO]] Run 'module load EasyBuild', and run 'eb --help' to get help on using EasyBuild.
[[INFO]] By default, EasyBuild will install software to $HOME/.local/easybuild.
[[INFO]] To install software with EasyBuild to /tmp/ebbootstraptest, make sure $EASYBUILD_INSTALLPATH is set accordingly.
[[INFO]] See https://github.com/hpcugent/easybuild/wiki/Configuration for details on configuring EasyBuild.

@boegel boegel mentioned this pull request Feb 24, 2014
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 24, 2014

@pforai, @stdweird: remarks addressed, final output is now like below;

did you guys test the new bootstrap script on your end?

[[INFO]] Done!
[[INFO]] 
[[INFO]] EasyBuild v1.11.0 was installed to /tmp/ebbootstraptest, so make sure your $MODULEPATH includes /tmp/ebbootstraptest/modules/all
[[INFO]] 
[[INFO]] Run 'module load EasyBuild', and run 'eb --help' to get help on using EasyBuild.
[[INFO]] Set $EASYBUILD_MODULES_TOOL to 'Lmod' to use the same modules tool as was used now.
[[INFO]] 
[[INFO]] By default, EasyBuild will install software to $HOME/.local/easybuild.
[[INFO]] To install software with EasyBuild to /tmp/ebbootstraptest, make sure $EASYBUILD_INSTALLPATH is set accordingly.
[[INFO]] See https://github.com/hpcugent/easybuild/wiki/Configuration for details on configuring EasyBuild.

Comment thread easybuild/scripts/bootstrap_eb.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please cleanup the output file

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

again, the tmpdir which is being passed down to this check_module_command function is cleaned up in the main function...

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 26, 2014

Remaining issue fixed, should be good to go now.

Thanks for the review @stdweird, @pforai!

boegel added a commit that referenced this pull request Feb 26, 2014
enhance bootstrap script to also support Lmod and modulecmd.tcl
@boegel boegel merged commit 53dcb06 into easybuilders:develop Feb 26, 2014
@boegel boegel deleted the enhance_bootstrap branch February 26, 2014 19:42
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.

3 participants