diff options
| author | Todd Gamblin <tgamblin@llnl.gov> | 2023-04-18 03:00:04 -0700 |
|---|---|---|
| committer | Todd Gamblin <tgamblin@llnl.gov> | 2023-04-18 03:00:04 -0700 |
| commit | d8a26905ee0fc9e72a05a87bf7ced7e7fc2d4900 (patch) | |
| tree | c13f8f2fbb2b0d94ff1aa995f6fbe068c17a1412 /.flake8 | |
| parent | 480b7f397e029a3de8d8a032170cd9e9862d64d3 (diff) | |
| download | spack-d8a26905ee0fc9e72a05a87bf7ced7e7fc2d4900.tar.gz spack-d8a26905ee0fc9e72a05a87bf7ced7e7fc2d4900.tar.bz2 spack-d8a26905ee0fc9e72a05a87bf7ced7e7fc2d4900.tar.xz spack-d8a26905ee0fc9e72a05a87bf7ced7e7fc2d4900.zip | |
refactor: unify use of spack.util.editor
Code from `spack.util.editor` was duplicated into our licensing hook in #11968. We
really only want one place where editor search logic is implemented. This consolidates
the logic into `spack.util.editor`, including a special case to run `gvim` with `-f`.
- [x] consolidate editor search logic in spack.util.editor
- [x] add tests for licensing case, where `Executable` is used instead of `os.execv`
- [x] make `_exec_func` argument of `editor()` into public `exec_fn` arg
- [x] add type annotations
Diffstat (limited to '.flake8')
0 files changed, 0 insertions, 0 deletions