u-boot.git/tools/patman/patchstream.py, branch v2016.09

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

patman: cover letter shows like 00/xx if more than 10 patches

2015-04-18T22:24:25+00:00

Make cover letter shows like 0/x, 00/xx and 000/xxx etc.

Signed-off-by: Josh Wu 
Acked-by: Simon Glass

patman: Check commit_match before stripping leading whitespace

2015-01-30T22:55:26+00:00

True commit lines start at column zero.  Anything that is indented
is part of the commit message instead.  I noticed this by trying to
run buildman with commit e3a4facdfc07179ebe017a07b8de6224a935a9f3
as master, which contained a reference to a Linux commit inside
the commit message.  ProcessLine saw that as a genuite commit
line, and thus buildman tried to build it, and died with an
exception because that SHA is not present in the U-Boot tree.

Signed-off-by: Scott Wood 
Acked-by: Simon Glass

patman: Use the full commit hash for 'git checkout'

2014-11-27T03:25:39+00:00

Even with the initial 8 characeters of the hash we will sometimes get a
collision. Use the full hash.

Signed-off-by: Simon Glass

buildman: Ignore conflicting tags

2014-09-09T22:38:31+00:00

Tags like Series-version are normally expected to appear once, and with a
unique value. But buildman doesn't actually look at these tags. So ignore
conflicts.

This allows bulidman to build a branch containing multiple patman series.

Reported-by: Steve Rae 
Signed-off-by: Simon Glass