Summary

Port remaining client backends to use run_process() instead of execute().

Review Request #14485 — Created June 27, 2025 and submitted Aug. 7, 2025, 8:38 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

This change ports the ClearCase and SOS client backends to use the new
run_process() method instead of execute(). Plastic is untouched per
Christian's request.

Testing Done

Ran unit tests.

Commits

Summary	ID
Port remaining client backends to use run_process() instead of execute(). This change ports the ClearCase and SOS client backends to use the new `run_process()` method instead of `execute()`. Plastic is untouched per Christian's request. Testing Done: Ran unit tests.	a052392adb912dc03a1a98ba0539724861f83408

Issues

Description	From	Last Updated
I have a bunch of work on Plastic that's going to conflict here. Can we leave that file with deprecated …	chipx86	July 10, 2025, 10:40 p.m.
Can we keep this to one keyword argument per line? It really helps with readability and any maintenance.	chipx86	July 10, 2025, 10:18 p.m.
Just a suggestion, but multi-line reads a lot better as: run_process([ '...', '...', '...', '...', ]) This would apply to …	chipx86	July 10, 2025, 10:18 p.m.

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

The issue has been resolved. Show all issues

I have a bunch of work on Plastic that's going to conflict here. Can we leave that file with deprecated code for now?

rbtools/clients/clearcase.py (Diff revision 1)

The issue has been resolved. Show all issues

Can we keep this to one keyword argument per line? It really helps with readability and any maintenance.

rbtools/clients/clearcase.py (Diff revision 1)

The issue has been resolved. Show all issues

Just a suggestion, but multi-line reads a lot better as:

run_process([
      '...', '...',
      '...', '...',
])


This would apply to many in this change.

Description:

~		This change ports the ClearCase, Plastic, and SOS client backends to use
~		the new `run_process()` method instead of `execute()`.
	~	This change ports the ClearCase and SOS client backends to use the new
	~	`run_process()` method instead of `execute()`. Plastic is untouched per
	+	Christian's request.

Commits:

	Summary	ID
	Port remaining client backends to use run_process() instead of execute(). This change ports the ClearCase, Plastic, and SOS client backends to use the new `run_process()` method instead of `execute()`. Testing Done: Ran unit tests.	14d8eec07767110f5460fce61b15e1a6d35e6bc7
	Port remaining client backends to use run_process() instead of execute(). This change ports the ClearCase and SOS client backends to use the new `run_process()` method instead of `execute()`. Plastic is untouched per Christian's request. Testing Done: Ran unit tests.	a052392adb912dc03a1a98ba0539724861f83408

Diff:

Revision 2 (+592 -336)

Show changes

	rbtools/clients/clearcase.py
	rbtools/clients/sos.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to master (f2dd94d)