diff options
| author | Harmen Stoppels <me@harmenstoppels.nl> | 2024-02-09 20:52:09 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-02-09 20:52:09 +0100 |
| commit | 4f0a8fce524fccadab392b35d3403945daae07fd (patch) | |
| tree | 76ea91d4d5892c13ca558dad8353872c8fb8cdfc /LICENSE-MIT | |
| parent | 7ff3b17f14e7d0c2033589e8b3e2f2afaa87c784 (diff) | |
| download | spack-4f0a8fce524fccadab392b35d3403945daae07fd.tar.gz spack-4f0a8fce524fccadab392b35d3403945daae07fd.tar.bz2 spack-4f0a8fce524fccadab392b35d3403945daae07fd.tar.xz spack-4f0a8fce524fccadab392b35d3403945daae07fd.zip | |
hooks: remove 7 unused hooks (#42575)
These 7 hooks were not used.
- Six of them related to install phases were unused after `spack`
`monitor` was removed, and the code seems to have bit rotten as there
were reports they were not (always?) triggered when they should.
- The post environment one was made redundant after spack install for
environment started following the common code path for generating
module files in #42147.
It should not be a breaking change to remove, since users cannot define
hooks in extensions, they would have to fork Spack.
If we ever _were_ to make those hooks extendable outside of core Spack,
it would also be better to start with fewer rather than more, cause
everything you expose gets relied upon...
Removing those also allows us to rethink what hooks we really need, and
in particular it seems like we need a hook that runs post install also when
the spec is inserted into the database.
Diffstat (limited to 'LICENSE-MIT')
0 files changed, 0 insertions, 0 deletions