[vim/vim] Vim9: cannot use "system()" without "{input}" optional argument (#8650)

lacygoill

unread,

Jul 27, 2021, 4:24:29 PM7/27/21

to vim/vim, Subscribed

Describe the bug

In Vim9 script, we cannot use system() to start an external process, without the optional argument {input}.

To Reproduce

Run this shell command:

vim -Nu NONE +'vim9 system(":")'

E1224 is given:

E1224: String or List required for argument 2

Expected behavior

No error is given, because we didn't pass the 2nd optional argument.

Environment

Vim version: 8.2 Included patches: 1-3229
OS: Ubuntu 20.04.2 LTS
Terminal: XTerm(353)

Additional context

Regression introduced in 8.2.3229.

Same issue with systemlist():

vim9script
systemlist(':')

E1224: String or List required for argument 2

cc @yegappan via #8646

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yegappan Lakshmanan

unread,

Jul 28, 2021, 12:30:02 AM7/28/21

to vim_dev, reply+ACY5DGBW2CZL24C7X7...@reply.github.com, vim/vim, Subscribed

Hi,

On Tue, Jul 27, 2021 at 1:24 PM lacygoill <vim-dev...@256bit.org> wrote:

Describe the bug

In Vim9 script, we cannot use system() to start an external process, without the optional argument {input}.

To Reproduce

Run this shell command:
vim -Nu NONE +'vim9 system(":")'
E1224 is given:
E1224: String or List required for argument 2
Expected behavior

No error is given, because we didn't pass the 2nd optional argument.

Environment

Vim version: 8.2 Included patches: 1-3229

OS: Ubuntu 20.04.2 LTS

Terminal: XTerm(353)

Additional context

Regression introduced in 8.2.3229.

Same issue with systemlist():
vim9script
systemlist(':')
E1224: String or List required for argument 2
cc @yegappan via #8646

Thanks for reporting the issue. I have created PR #8651 with

a fix for this. Let me know if you see any other issues with other

functions.

- Yegappan

vim-dev ML

unread,

Jul 28, 2021, 12:30:23 AM7/28/21

to vim/vim, vim-dev ML, Your activity

Hi,

On Tue, Jul 27, 2021 at 1:24 PM lacygoill ***@***.***> wrote:

> *Describe the bug*

>
> In Vim9 script, we cannot use system() to start an external process,
> without the optional argument {input}.
>

> *To Reproduce*

>
> Run this shell command:
>
> vim -Nu NONE +'vim9 system(":")'
>
> E1224 is given:
>
> E1224: String or List required for argument 2
>

> *Expected behavior*

>
> No error is given, because we didn't pass the 2nd optional argument.
>

> *Environment*
>
> - Vim version: 8.2 Included patches: 1-3229
> - OS: Ubuntu 20.04.2 LTS
> - Terminal: XTerm(353)
>
> *Additional context*
>
> Regression introduced in 8.2.3229.
> ------------------------------
>
> Same issue with systemlist():
>
> vim9scriptsystemlist(':')

>
> E1224: String or List required for argument 2
>

> ------------------------------
>
> cc @yegappan <https://github.com/yegappan> via #8646
> <https://github.com/vim/vim/pull/8646>

>
>
>
Thanks for reporting the issue. I have created PR #8651 with
a fix for this. Let me know if you see any other issues with other
functions.

- Yegappan

Bram Moolenaar

unread,

Jul 28, 2021, 5:52:19 AM7/28/21

to vim/vim, vim-dev ML, Comment

Closed #8650 via 7e6a2a6.

—
You are receiving this because you commented.

Reply all

Reply to author

Forward