u-boot.git/tools/patman/patchstream.py, branch v2017.07

patman: Add a functional test

2017-06-09T02:21:59+00:00

The existing test (patman --test) only covers basic checkpatch output.
We have had some problems with unicode processing and could use test
coverage for the various tags patman supports.

Add a new functional test which runs most of the patman flow on a few
test commits and checks that the results are correct.

See the documentation in the test for a description of what it does.

Signed-off-by: Simon Glass 
Tested-by: Philipp Tomsich

patman: Don't return the series in FixPatches()

2017-06-09T02:21:59+00:00

There is no need for this function to return the same object that was
passed in. Drop the return value.

Signed-off-by: Simon Glass 
Tested-by: Philipp Tomsich

patman: Don't report unicode character

2017-06-09T02:21:59+00:00

Unicode characters may appear in input patches so we should not warn about
them. Drop this warning.

Signed-off-by: Simon Glass 
Tested-by: Philipp Tomsich

patman: Make print statements python 3.x safe

2016-10-09T15:30:32+00:00

In python 3.x, print must be used as a function call. Convert all print
statements to the function call style, importing from __future__ where
we print with no trailing newline or print to a file object.

Signed-off-by: Paul Burton 
Acked-by: Simon Glass

tools: patman: Handle missing 'END' in non-last commit of a series

2016-07-11T20:06:44+00:00

The following python error:

Traceback (most recent call last):
  File "./tools/patman/patman", line 144, in 
    series = patchstream.FixPatches(series, args)
  File "./tools/patman/patchstream.py", line 477, in FixPatches
    commit = series.commits[count]
IndexError: list index out of range

is seen when:

- 'END' is missing in those tags
- those tags are put in the last part in a commit message
- the commit is not the last commit of the series

Add testing logic to see if a new commit starts.

Signed-off-by: Bin Meng 
Acked-by: Simon Glass

tools: patman: Handle missing blank line for 'Series-changes'

2016-07-11T20:06:44+00:00

'Series-changes' uses blank line to indicate its end. If that is
missing, series internal state variable 'in_change' may be wrong.
Correct its state.

Signed-off-by: Bin Meng 
Acked-by: Simon Glass

tools: patman: Generate cover letter correctly when 'END' is missing

2016-07-11T20:06:44+00:00

If 'END' is missing in a 'Cover-letter' section, and that section
happens to show up at the very end of the commit message, and the
commit is the last commit of the series, patman fails to generate
cover letter for us. Handle this in CloseCommit of patchstream.

Signed-off-by: Bin Meng 
Acked-by: Simon Glass

tools: patman: Handle tag sections without an 'END'

2016-07-11T20:06:44+00:00

'Cover-letter', 'Series-notes' and 'Commit-notes' tags require an
'END' to be put at the end of its section. If we forget to put an
'END' in those sections, and these sections are followed by another
patman tag, patman generates incorrect patches. This adds codes to
handle such scenario.

Signed-off-by: Bin Meng 
Acked-by: Simon Glass

tools: patman: Use cover_match for 'Cover-letter'

2016-07-11T20:06:44+00:00

Like other patman tags, use a new variable cover_match to indicate
a match for 'Cover-letter'.

Signed-off-by: Bin Meng 
Acked-by: Simon Glass

patman: Add a missing space in GetMetaDataForList()

2016-03-14T21:34:50+00:00

Fix this nit to keep the code consistent.

Signed-off-by: Simon Glass 
Reviewed-by: Joe Hershberger