Assume DESI_ROOT is external

desihub / desitree

Describes the DESI directory hierarchy and maps it to environment variables.

BSD 3-Clause "New" or "Revised" License

0 stars 0 forks source link

Assume DESI_ROOT is external #7

Closed weaverba137 closed 2 years ago

weaverba137 commented 3 years ago

This PR fixes #6 by not defining DESI_ROOT within the module itself. As a side-effect it allows this module to work anywhere that DESI_ROOT is defined, not just NERSC.

As far as I can tell, desitree is never used in a main/master checkout, but only as a tag, so we can go ahead and merge and tag this, then later incorporate the latest tag into a desimodules update.

weaverba137 commented 3 years ago

Pinging @sbailey for review.

weaverba137 commented 2 years ago

I'm going to go ahead and merge this so I can test 0.6.0 on perlmutter.