diff options
author | John W. Parent <45471568+johnwparent@users.noreply.github.com> | 2024-10-18 16:36:16 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-10-18 13:36:16 -0700 |
commit | 31cfcafeba35429d9b6d811ae27eff0e88b1fee8 (patch) | |
tree | 0629e837ba391c1c11417bf9084a219ba64eb3bf /.flake8 | |
parent | 230bc7010a5bbcfd213c2531be91ebc7381d3886 (diff) | |
download | spack-31cfcafeba35429d9b6d811ae27eff0e88b1fee8.tar.gz spack-31cfcafeba35429d9b6d811ae27eff0e88b1fee8.tar.bz2 spack-31cfcafeba35429d9b6d811ae27eff0e88b1fee8.tar.xz spack-31cfcafeba35429d9b6d811ae27eff0e88b1fee8.zip |
Build logic fix: reorder definition of package module variables (#46992)
#44327 made sure to always run `set_package_py_globals` on all
packages before running `setup_dependent_package` for any package,
so that packages implementing the latter could depend on variables
like `spack_cc` being defined.
This ran into an undocumented dependency: `std_cmake_args` is set in
`set_package_py_globals` and makes use of `cmake_prefix_paths` (if it
is defined in the package); `py-torch`es implementation of
`cmake_prefix_paths` depends on a variable set by
`setup_dependent_package` (`python_platlib`).
This generally restores #44327, and corrects the resulting issue by
moving assignment of `std_cmake_args` to after both actions have been
run.
Diffstat (limited to '.flake8')
0 files changed, 0 insertions, 0 deletions