click.edit() expects editor to be executable with no args

[pallets]

click.edit() expects editor to be executable with no args

[mitsuhiko]

To the best of my knowledge EDITOR does not support arguments. If you can point me to a piece of documentation that claims otherwise please reopen this issue.

[untitaker]

afaik kabniel is not able to reopen the issue once you close it, @mitsuhiko

[untitaker]

I believe i have seen the presence of arguments in the EDITOR variable several times when reading through the documentation of Linux distribution. I believe this is similar to the situation with shebangs. However, is there a reason why it should not be allowed? I am aware that the splitting of such a value goes beyond str.split(), but it still shouldn't be too hard.

[mitsuhiko]

Because the behavior is different. If EDITOR accepts arguments then the editor name needs quoting if it's installed into a folder with whitespace which is very common on windows (C:\Program files).

[mitsuhiko]

I changed that now as this is apparently what git and hg do.

[dufferzafar]

Has this been fixed? I'm facing a similar issue when I try to use VS Code as my editor:

>>> click.edit(editor='code-insiders -w')

/bin/sh: code-insiders -w: command not found

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.8/site-packages/click/termui.py", line 587, in edit
    return editor.edit(text)
  File "/usr/lib/python3.8/site-packages/click/_termui_impl.py", line 497, in edit
    self.edit_file(name)
  File "/usr/lib/python3.8/site-packages/click/_termui_impl.py", line 472, in edit_file
    raise ClickException("{}: Editing failed!".format(editor))
click.exceptions.ClickException: code-insiders -w: Editing failed!