Summary:

Fix rbt patch svn to set results_unicode=False

Review Request #12372 — Created June 14, 2022 and updated Aug. 11, 2022, 8:02 p.m.

Information
Owner:	rpowell
Repository:	RBTools
Branch:	master
Bugs:	4973
Depends On:
Reviewers
Groups:	rbtools
People:

Description

After applying an SVN patch, attempting to print the output could result in

CRITICAL: a bytes-like object is required, not 'str'

as the patch output was of type str.
This change calls the underlying rbtools.utils.process.execute function with the results_unicode=False similar to the git implementation (https://github.com/reviewboard/rbtools/blob/master/rbtools/clients/git.py#L1110) as suggested by  dushara

Testing Done

Tested locally with an SVN patch, confirmed the patch could apply successfully.

Commits

Summary	Author
Fix rbt patch for SVN expecting bytes output	Bob Powell

Screenshots

Issues

Description	From	Last Updated
Thanks for the patch! This looks like a symptom of a problem. patch_output should always be bytes, or problems can …	chipx86	Aug. 11, 2022, 7:54 p.m.

flake8 passed.

JSHint passed.

Testing Done:

Tested locally with an SVN patch, confirmed the patch could apply successfully.

Show all issues

Thanks for the patch!

This looks like a symptom of a problem. patch_output should always be bytes, or problems can occur elsewhere. It'll be important to find out why these are becoming str instead.

Change Summary:

update diff with dushara's suggestion on https://hellosplat.com/s/beanbag/tickets/4973/

Commits:

	Summary		Author
-		Fix rbt patch to print string output with stdout instead of stdout_bytes	Bob Powell
+		Fix rbt patch for SVN expecting bytes output	Bob Powell

Diff:

Revision 2 (+4 -2)

Show changes

rbtools/clients/svn.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Description:

		After applying an SVN patch, attempting to print the output could result in
		CRITICAL: a bytes-like object is required, not 'str'
		as the patch output was of type str.

~		This change calls the stdout.write function when patch_output is of type str.
	~	This change calls the underlying rbtools.utils.process.execute function with the results_unicode=False similar to the git implementation (https://github.com/reviewboard/rbtools/blob/master/rbtools/clients/git.py#L1110) as suggested by dushara

Summary:

-	Fix rbt patch to print string output with stdout instead of stdout_bytes
+	Fix rbt patch svn to set results_unicode=False