diff options
author | Peter Scheibel <scheibel1@llnl.gov> | 2020-11-12 12:26:23 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-12 12:26:23 -0800 |
commit | bb42470211a82fc333d48ba56c0d0990cd1faec5 (patch) | |
tree | 2bf23bb4ed77e2d9462cdb0b46561a0dd9563b95 /.github/actions/add-maintainers-as-reviewers/Dockerfile | |
parent | 81cab3b2711f12d82f73db44438afd5794529f1e (diff) | |
download | spack-bb42470211a82fc333d48ba56c0d0990cd1faec5.tar.gz spack-bb42470211a82fc333d48ba56c0d0990cd1faec5.tar.bz2 spack-bb42470211a82fc333d48ba56c0d0990cd1faec5.tar.xz spack-bb42470211a82fc333d48ba56c0d0990cd1faec5.zip |
macos: update build process to use spawn instead of fork (#18205)
Spack creates a separate process to do package installation. Different
operating systems and Python versions use different methods to create
it but up until Python 3.8 both Linux and Mac OS used "fork" (which
duplicates process memory, file descriptor table, etc.).
Python >= 3.8 on Mac OS prefers creating an entirely new process
(referred to as the "spawn" start method) because "fork" was found to
cause issues (in other words "spawn" is the default start method used
by multiprocessing.Process). Spack was dependent on the particular
behavior of fork to replicate process memory and transmit file
descriptors.
This PR refactors the Spack internals to support starting a child
process with the "spawn" method. To achieve this, it makes the
following changes:
- ensure that the package repository and other global state are
transmitted to the child process
- ensure that file descriptors are transmitted to the child process in
a way that works with multiprocessing and spawn
- make all the state needed for the build process and tests picklable
(package, stage, etc.)
- move a number of locally-defined functions into global scope so that
they can be pickled
- rework tests where needed to avoid using local functions
This PR also reworks sbang tests to work on macOS, where temporary
directories are deeper than the Linux sbang limit. We make the limit
platform-dependent (macOS supports 512-character shebangs)
See: #14102
Diffstat (limited to '.github/actions/add-maintainers-as-reviewers/Dockerfile')
0 files changed, 0 insertions, 0 deletions