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

define DESI_PRODUCT_ROOT #2

Closed sbailey closed 8 years ago

sbailey commented 8 years ago

It seems that "module load desitree" should define $DESI_PRODUCT_ROOT, but it does not. I see an old svn commit entry that specifically disabled this:

https://desi.lbl.gov/trac/changeset/837/code

But why? I was expecting this to be set when I tried running "desiInstall" and it complained that $DESI_PRODUCT_ROOT wasn't set.

weaverba137 commented 8 years ago

That's because DESI_PRODUCT_ROOT is only meant to be defined on non-NERSC systems. On NERSC systems, /project/projectdirs/desi/software/$NERSC_HOST is automatically detected.

What specific package is requiring the existence of $DESI_PRODUCT_ROOT? Remember, we are no longer supporting Modules on non-NERSC systems, and since desitree is literally only a Module file, I guess this issue must be related to a NERSC install. But again, the NERSC specific case is already handled in every product that follows the template. So I'm puzzled by this request.

weaverba137 commented 8 years ago

PS, the message I think you saw from desiInstall is only a warning, and can safely be ignored on NERSC systems. This message will probably be disabled in a future release of desiutil.

weaverba137 commented 8 years ago

I'm going to assume that this question has been answered.