diff options
author | Todd Gamblin <tgamblin@llnl.gov> | 2022-12-28 00:44:11 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-12-28 00:44:11 -0800 |
commit | 5f8c706128387b1258bd65010e4e06ef662283d9 (patch) | |
tree | e276deb0dd662cecfd0a005efd44c7d88f1ec1d9 /.github/CONTRIBUTING.md | |
parent | 558695793fcde4f1ef9f75f6c0ca9e41e5420cf6 (diff) | |
download | spack-5f8c706128387b1258bd65010e4e06ef662283d9.tar.gz spack-5f8c706128387b1258bd65010e4e06ef662283d9.tar.bz2 spack-5f8c706128387b1258bd65010e4e06ef662283d9.tar.xz spack-5f8c706128387b1258bd65010e4e06ef662283d9.zip |
Consolidate how Spack uses `git` (#34700)
Local `git` tests will fail with `fatal: transport 'file' not allowed` when using git 2.38.1 or higher, due to a fix for `CVE-2022-39253`.
This was fixed in CI in #33429, but that doesn't help the issue for anyone's local environment. Instead of fixing this with git config in CI, we should ensure that the tests run anywhere.
- [x] Introduce `spack.util.git`.
- [x] Use `spack.util.git.get_git()` to get a git executable, instead of `which("git")` everywhere.
- [x] Make all `git` tests use a `git` fixture that goes through `spack.util.git.get_git()`.
- [x] Add `-c protocol.file.allow=always` to all `git` invocations under `pytest`.
- [x] Revert changes from #33429, which are no longer needed.
Diffstat (limited to '.github/CONTRIBUTING.md')
0 files changed, 0 insertions, 0 deletions