[U-Boot] [PATCH v2 00/10] patman: Fix unicode operation and add a functional test

Dr. Philipp Tomsich philipp.tomsich at theobroma-systems.com
Mon Jun 5 22:50:45 UTC 2017


I have this on my tree since you sent it out and my last round of
patch-submissions was already done with this version of patman.
There were a few UTF-8 characters involved and all worked as
expected.

So you can consider it
	Tested-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com <mailto:philipp.tomsich at theobroma-systems.com>>

Cheers,
Phil.

> On 05 Jun 2017, at 23:41, Simon Glass <sjg at chromium.org> wrote:
> 
> Hi,
> 
> On 29 May 2017 at 15:31, Simon Glass <sjg at chromium.org> wrote:
>> With more recent versions of Python 2.7 (and perhaps patman itself)
>> something has changed such that patman sometimes reports unicode decode
>> errors, particularly when getting a list of maintainers for code covered
>> by a patch.
>> 
>> This series attempts to correct this. It also adds a functional test so
>> that operation can quickly be verified. This new test tests much more of
>> the code than the existing unit tests.
>> 
>> Changes in v2:
>> - Expand the patch to cover all cases
>> - Drop RFC tag
>> - Rewrite commit message
>> - Add new clean-up patches and a functional test
>> 
>> Philipp Tomsich (1):
>>  patman: encode CC list to UTF-8
>> 
>> Simon Glass (9):
>>  patman: Adjust handling of unicode email address
>>  patman: Don't convert input data to unicode
>>  patman: Rename 'str' variable in EmailPatches()
>>  patman: Don't report unicode character
>>  patman: Don't return the series in FixPatches()
>>  patman: Add unicode to test patches
>>  patman: Add a maintainer test feature to MakeCcFile()
>>  patman: Rename 'list' variable in MakeCcFile()
>>  patman: Add a functional test
>> 
>> tools/patman/cros_subprocess.py                    |   4 -
>> tools/patman/func_test.py                          | 242 +++++++++++++++++++++
>> tools/patman/gitutil.py                            |   8 +-
>> tools/patman/patchstream.py                        |  23 +-
>> tools/patman/patman.py                             |  10 +-
>> tools/patman/series.py                             |  28 ++-
>> tools/patman/test.py                               |   9 +
>> tools/patman/test/0000-cover-letter.patch          |  23 ++
>> .../test/0001-pci-Correct-cast-for-sandbox.patch   |  48 ++++
>> ...-cast-for-sandbox-in-fdtdec_setup_memory_.patch |  73 +++++++
>> tools/patman/test/test01.txt                       |  56 +++++
>> 11 files changed, 493 insertions(+), 31 deletions(-)
>> create mode 100644 tools/patman/func_test.py
>> create mode 100644 tools/patman/test/0000-cover-letter.patch
>> create mode 100644 tools/patman/test/0001-pci-Correct-cast-for-sandbox.patch
>> create mode 100644 tools/patman/test/0002-fdt-Correct-cast-for-sandbox-in-fdtdec_setup_memory_.patch
>> create mode 100644 tools/patman/test/test01.txt
>> 
>> --
>> 2.13.0.219.gdb65acc882-goog
>> 
> 
> Any comments on this one? I'm tempted to apply it to get people testing it!
> 
> Regards,
> Simon



More information about the U-Boot mailing list