diff options
author | Matt Belhorn <belhornmp@ornl.gov> | 2016-10-31 00:02:27 -0400 |
---|---|---|
committer | Todd Gamblin <tgamblin@llnl.gov> | 2016-10-30 21:02:27 -0700 |
commit | 7ccf56a0b72fc6ce550d10e125a0dfa99d8b5f51 (patch) | |
tree | d3092d5780c0f6c6f38e053a945d91e3eefb93d8 /lib/spack/spack/config.py | |
parent | a714377123cf2aefb895badfbf9d4baf81b39654 (diff) | |
download | spack-7ccf56a0b72fc6ce550d10e125a0dfa99d8b5f51.tar.gz spack-7ccf56a0b72fc6ce550d10e125a0dfa99d8b5f51.tar.bz2 spack-7ccf56a0b72fc6ce550d10e125a0dfa99d8b5f51.tar.xz spack-7ccf56a0b72fc6ce550d10e125a0dfa99d8b5f51.zip |
Removes cyclic dependency on spack.config. (#2121)
Merge #2030 added a cyclic dependency between the Cray platform needing
to read a `targets.yaml` config file and `config.py` needing to get the
platform names.
This commit removes the cyclic dependency in favor of the more general
config scheme. It also removes the now functionless `targets.yaml`
config file. This breaks 'frontend' targets on the Cray platform but
all architecture targets, including the frontend, that are provided by
CrayPE are added to the Platform anyway so users can be explicit about
the architecture targeted by the Cray compiler wrappers:
```
spack spec libelf arch=cray-CNL-frontend
```
becomes
```
spack spec libelf arch=cray-CNL-mc8 # on an XK7 or
spack spec libelf arch=cray-CNL-sandybridge # on an older XC30, etc..
```
The only way the 'frontend' target can be defined after this commit is
through target environment variables.
Diffstat (limited to 'lib/spack/spack/config.py')
-rw-r--r-- | lib/spack/spack/config.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/spack/spack/config.py b/lib/spack/spack/config.py index 6a67f01d66..de5f55775c 100644 --- a/lib/spack/spack/config.py +++ b/lib/spack/spack/config.py @@ -179,7 +179,6 @@ section_schemas = { 'mirrors': spack.schema.mirrors.schema, 'repos': spack.schema.repos.schema, 'packages': spack.schema.packages.schema, - 'targets': spack.schema.targets.schema, 'modules': spack.schema.modules.schema, } |