Summary

Fix subprocess.run() arguments with input_string parameter.

Review Request #14571 — Created Aug. 25, 2025 and submitted Sept. 4, 2025, 5:22 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

The newly-added input_string parameter to run_process could end up
passing both stdin=subprocess.PIPE and input=..., which causes an
error. This change makes it so we don't do that.

Testing Done

Used some code that passed input_string in to run_process and no
longer saw the error.

Commits

Summary	ID
Fix subprocess.run() arguments with input_string parameter. The newly-added `input_string` parameter to `run_process` could end up passing both `stdin=subprocess.PIPE` and `input=...`, which causes an error. This change makes it so we don't do that. Testing Done: Used some code that passed `input_string` in to `run_process` and no longer saw the error.	ksoklvlvmoywmkkntvzxzrokusmnzomq

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (07a2507)