diff options
| author | Todd Gamblin <tgamblin@llnl.gov> | 2023-04-18 03:00:04 -0700 |
|---|---|---|
| committer | Todd Gamblin <tgamblin@llnl.gov> | 2023-04-18 16:23:00 -0700 |
| commit | 2f30da176228c7a78447f6894e8406ed7b7ff4dd (patch) | |
| tree | ae92df4f5615c3ba556cb900610940b7a1530bb8 /etc | |
| parent | d92c21ec9709589e20abdd4eaadcda0697a594ed (diff) | |
| download | spack-2f30da176228c7a78447f6894e8406ed7b7ff4dd.tar.gz spack-2f30da176228c7a78447f6894e8406ed7b7ff4dd.tar.bz2 spack-2f30da176228c7a78447f6894e8406ed7b7ff4dd.tar.xz spack-2f30da176228c7a78447f6894e8406ed7b7ff4dd.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 'etc')
0 files changed, 0 insertions, 0 deletions