payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
18 stars 26 forks source link

Check for MODULESHOME environment variable for gadi support #210

Closed aidanheerdegen closed 4 years ago

aidanheerdegen commented 4 years ago

Fixes https://github.com/payu-org/payu/issues/209

On gadi MODULE_VERSION environment variable is no longer set by default, but MODULESHOME is. This change checks for MODULESHOME before trying to generate it from MODULE_VERSION.

aidanheerdegen commented 4 years ago

AsMODULE_VERSION is set on raijin it is safe to change the default to match gadi in any case.

I have tested this works on raijin

aidanheerdegen commented 4 years ago

If there are no objections I'll merge, tag a new version and push it to conda/analysis3-unstable

aidanheerdegen commented 4 years ago

I'll just merge I think, as there could be others testing gadi who will be tripped up by this.