Summary

Add Jujutsu binary file patching.

Review Request #14720 — Created Dec. 1, 2025 and submitted Jan. 9, 2026, 12:20 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

Jujutsu actually doesn't require anything different from the base
patcher support for patching binary files, because it doesn't require
any manual staging or custom commands for adding/moving/deleting files.
This change therefore just adds unit tests for the binary file patching
functionality.

Testing Done

Ran unit tests.

Commits

Summary	ID
Add Jujutsu binary file patching. Jujutsu actually doesn't require anything different from the base patcher support for patching binary files, because it doesn't require any manual staging or custom commands for adding/moving/deleting files. This change therefore just adds unit tests for the binary file patching functionality. Testing Done: Ran unit tests.	nkmsuxoonqrvzqmvwuluqkzvmqrxmutp

Issues

Description	From	Last Updated
I feel like this pattern is common enough and strange-looking enough that it should probably become a utility function.	chipx86	Dec. 4, 2025, 9:53 a.m.

flake8 passed.

JSHint passed.

Summary:: Add binary file patching for Jujutsu
Add Jujutsu binary file patching.

rbtools/clients/tests/test_jujutsu.py (Diff revision 1)

The issue has been resolved. Show all issues

I feel like this pattern is common enough and strange-looking enough that it should probably become a utility function.

david Dec. 4, 2025, 9:39 a.m.

Sure. The actual implementation of the utility function will end up in the patcher change due to the stack of the commits.

Change Summary:

Use new make_binary_file_patch helper.

Commits:

	Summary	ID
	Add binary file patching for Jujutsu Jujutsu actually doesn't require anything different from the base patcher support for patching binary files, because it doesn't require any manual staging or custom commands for adding/moving/deleting files. This change therefore just adds unit tests for the binary file patching functionality. Testing Done: Ran unit tests.	nkmsuxoonqrvzqmvwuluqkzvmqrxmutp
	Add Jujutsu binary file patching. Jujutsu actually doesn't require anything different from the base patcher support for patching binary files, because it doesn't require any manual staging or custom commands for adding/moving/deleting files. This change therefore just adds unit tests for the binary file patching functionality. Testing Done: Ran unit tests.	nkmsuxoonqrvzqmvwuluqkzvmqrxmutp

Diff:

Revision 2 (+1150 -2)

Show changes

rbtools/clients/tests/test_jujutsu.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

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