[vim/vim] fix: system() List arg link error without FEAT_JOB_CHANNEL (PR #19826)

[mattn]

Move build_argv_from_list() and mch_get_cmd_output_direct() out of FEAT_JOB_CHANNEL preprocessor guards so that system() with a List argument (patch 9.2.0250) builds correctly in configurations that have FEAT_EVAL but not FEAT_JOB_CHANNEL (e.g. FEAT_NORMAL without GUI on Windows).

Thanks to John Marriott (@basilisk0315) for reporting and providing initial patches.

---

You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/19826

Commit Summary

• 1f4b4e0 fix: build_argv_from_list() and mch_get_cmd_output_direct() link error without FEAT_JOB_CHANNEL

File Changes

(2 files)

• M src/misc2.c (5)
• M src/os_win32.c (7)

Patch Links:

• https://github.com/vim/vim/pull/19826.patch
• https://github.com/vim/vim/pull/19826.diff

—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19826@github.com>

[John Marriott]

basilisk0315 left a comment (vim/vim#19826)

Applying this patch fixes it for me.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19826/c4131297949@github.com>

[Christian Brabandt]

chrisbra left a comment (vim/vim#19826)

thanks

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19826/c4137680229@github.com>

[Christian Brabandt]

Closed #19826 via 68b3585.

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/19826/issue_event/23947448368@github.com>