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

Simon Glass sjg at chromium.org
Mon Jun 5 21:41:31 UTC 2017


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