Summary

Add SVN binary file patching.

Review Request #14717 — Created Nov. 29, 2025 and updated Nov. 29, 2025, 11:55 a.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

This change adds support to the SVNPatcher for applying binary files
as part of changes. Things have been reorganized a little bit in order
to make it easier to deal with both empty files and binary files, which
appear similarly in the diff.

Additionally, several of our tests had index lines that did not start
with a /, which was inconsistent with real-world data. I've fixed
these up to match.

Testing Done


Ran unit tests.
Used rbt patch with a bunch of changes that did various file

  operations on binary files.

Commits

Summary	ID
Add SVN binary file patching. This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.	xlmyyqprwnolxknqmzqskqkvpurszzwy

Summary

Add SVN binary file patching.

This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.

xlmyyqprwnolxknqmzqskqkvpurszzwy

Issues

Description	From	Last Updated
continuation line over-indented for hanging indent Column: 29 Error code: E126	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (96 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (92 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (92 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (92 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (89 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:53 a.m.
line too long (84 > 79 characters) Column: 80 Error code: E501	reviewbot	Nov. 29, 2025, 11:55 a.m.

flake8 failed.

JSHint passed.

flake8

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

The issue has been dropped. Show all issues

continuation line over-indented for hanging indent

Column: 29
Error code: E126

rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (96 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (92 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (92 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (92 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (89 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```
rbtools/clients/tests/test_svn.py (Diff revision 1)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```

Commits:

	Summary	ID
	Add SVN binary file patching. This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.	xlmyyqprwnolxknqmzqskqkvpurszzwy
	Add SVN binary file patching. This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.	xlmyyqprwnolxknqmzqskqkvpurszzwy

Diff:

Revision 2 (+2134 -454)

Show changes

	rbtools/clients/svn.py
	rbtools/clients/tests/test_svn.py

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

rbtools/clients/tests/test_svn.py (Diff revision 2)
The issue has been dropped. Show all issues
```
line too long (84 > 79 characters)

Column: 80
Error code: E501
```

Commits:

	Summary	ID
	Add SVN binary file patching. This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.	xlmyyqprwnolxknqmzqskqkvpurszzwy
	Add SVN binary file patching. This change adds support to the `SVNPatcher` for applying binary files as part of changes. Things have been reorganized a little bit in order to make it easier to deal with both empty files and binary files, which appear similarly in the diff. Additionally, several of our tests had index lines that did not start with a `/`, which was inconsistent with real-world data. I've fixed these up to match. Testing Done: - Ran unit tests. - Used `rbt patch` with a bunch of changes that did various file operations on binary files.	xlmyyqprwnolxknqmzqskqkvpurszzwy

Diff:

Revision 3 (+2136 -454)

Show changes

	rbtools/clients/svn.py
	rbtools/clients/tests/test_svn.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Description:: This change adds support to the SVNPatcher for applying binary files
as part of changes. Things have been reorganized a little bit in order
to make it easier to deal with both empty files and binary files, which
appear similarly in the diff.

Additionally, several of our tests had index lines that did not start
with a /, which was inconsistent with real-world data. I've fixed
these up to match.
-
-
Testing Done:
- - Ran unit tests.
- - Used rbt patch with a bunch of changes that did various file
- operations on binary files.
Testing Done:: +
Ran unit tests.
+
Used rbt patch with a bunch of changes that did various file
operations on binary files.

		This change adds support to the `SVNPatcher` for applying binary files
		as part of changes. Things have been reorganized a little bit in order
		to make it easier to deal with both empty files and binary files, which
		appear similarly in the diff.

		Additionally, several of our tests had index lines that did not start
		with a `/`, which was inconsistent with real-world data. I've fixed
		these up to match.
-
-		Testing Done:
-		- Ran unit tests.
-		- Used `rbt patch` with a bunch of changes that did various file
-		operations on binary files.

	+	Ran unit tests.
	+	Used `rbt patch` with a bunch of changes that did various file operations on binary files.