]> git.ipfire.org Git - thirdparty/vim.git/commit
runtime(dist): centralize safe executable check and add vim library (#13413)
authorD. Ben Knoble <ben.knoble+github@gmail.com>
Sat, 4 Nov 2023 09:11:17 +0000 (05:11 -0400)
committerGitHub <noreply@github.com>
Sat, 4 Nov 2023 09:11:17 +0000 (10:11 +0100)
commitcd8a3eaf5348feacfecab4b374b7ea4ce6a97422
tree93c6255071557c69bc8e15ee7830a31c0a3a7149
parent08b1c61e8b67be2b6a95d9b85d3bbf84cc70712f
runtime(dist): centralize safe executable check and add vim library (#13413)

Follow up to 816fbcc26 (patch 9.0.1833: [security] runtime file fixes,
2023-08-31) and f7ac0ef50 (runtime: don't execute external commands when
loading ftplugins, 2023-09-06).

This puts the logic for safe executable checks in a single place, by introducing
a central vim library, so all filetypes benefit from consistency.

Notable changes:
- dist#vim because the (autoload) namespace for a new runtime support
  library. Supporting functions should get documentation. It might make
  life easier for NeoVim devs to make the documentation a new file
  rather than cram it into existing files, though we may want
  cross-references to it somewhere…
- The gzip and zip plugins need to be opted into by enabling execution
  of those programs (or the global plugin_exec). This needs
  documentation or discussion.
- This fixes a bug in the zig plugin: code setting s:tmp_cwd was removed
  in f7ac0ef50 (runtime: don't execute external commands when loading
  ftplugins, 2023-09-06), but the variable was still referenced. Since
  the new function takes care of that automatically, the variable is no
  longer needed.

Signed-off-by: D. Ben Knoble <ben.knoble+github@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
runtime/autoload/dist/vim.vim [new file with mode: 0644]
runtime/autoload/gzip.vim
runtime/autoload/zip.vim
runtime/ftplugin/awk.vim
runtime/ftplugin/changelog.vim
runtime/ftplugin/perl.vim
runtime/ftplugin/zig.vim