From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dr. Philipp Tomsich Date: Tue, 6 Jun 2017 00:50:45 +0200 Subject: [U-Boot] [PATCH v2 00/10] patman: Fix unicode operation and add a functional test In-Reply-To: References: <20170529213132.21217-1-sjg@chromium.org> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > Cheers, Phil. > On 05 Jun 2017, at 23:41, Simon Glass wrote: > > Hi, > > On 29 May 2017 at 15:31, Simon Glass 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