<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/tools/patman/patchstream.py, branch v2018.07</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/tools/patman/patchstream.py?h=v2018.07</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/tools/patman/patchstream.py?h=v2018.07'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2018-05-07T13:34:12Z</updated>
<entry>
<title>SPDX: Convert all of our single license tags to Linux Kernel style</title>
<updated>2018-05-07T13:34:12Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2018-05-06T21:58:06Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=83d290c56fab2d38cd1ab4c4cc7099559c1d5046'/>
<id>urn:sha1:83d290c56fab2d38cd1ab4c4cc7099559c1d5046</id>
<content type='text'>
When U-Boot started using SPDX tags we were among the early adopters and
there weren't a lot of other examples to borrow from.  So we picked the
area of the file that usually had a full license text and replaced it
with an appropriate SPDX-License-Identifier: entry.  Since then, the
Linux Kernel has adopted SPDX tags and they place it as the very first
line in a file (except where shebangs are used, then it's second line)
and with slightly different comment styles than us.

In part due to community overlap, in part due to better tag visibility
and in part for other minor reasons, switch over to that style.

This commit changes all instances where we have a single declared
license in the tag as both the before and after are identical in tag
contents.  There's also a few places where I found we did not have a tag
and have introduced one.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>patman: Add a functional test</title>
<updated>2017-06-09T02:21:59Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2017-05-29T21:31:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6e87ae1c0730ec99b6824c2a3dc8510813f92b98'/>
<id>urn:sha1:6e87ae1c0730ec99b6824c2a3dc8510813f92b98</id>
<content type='text'>
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 &lt;sjg@chromium.org&gt;
Tested-by: Philipp Tomsich &lt;philipp.tomsich@theobroma-systems.com&gt;
</content>
</entry>
<entry>
<title>patman: Don't return the series in FixPatches()</title>
<updated>2017-06-09T02:21:59Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2017-05-29T21:31:27Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=db116cc8d095ec499ae748ea864fa0def54ca3d6'/>
<id>urn:sha1:db116cc8d095ec499ae748ea864fa0def54ca3d6</id>
<content type='text'>
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 &lt;sjg@chromium.org&gt;
Tested-by: Philipp Tomsich &lt;philipp.tomsich@theobroma-systems.com&gt;
</content>
</entry>
<entry>
<title>patman: Don't report unicode character</title>
<updated>2017-06-09T02:21:59Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2017-05-29T21:31:26Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=04f7870635dff90ddc1b8465b5b8304c2d429a92'/>
<id>urn:sha1:04f7870635dff90ddc1b8465b5b8304c2d429a92</id>
<content type='text'>
Unicode characters may appear in input patches so we should not warn about
them. Drop this warning.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Tested-by: Philipp Tomsich &lt;philipp.tomsich@theobroma-systems.com&gt;
</content>
</entry>
<entry>
<title>patman: Make print statements python 3.x safe</title>
<updated>2016-10-09T15:30:32Z</updated>
<author>
<name>Paul Burton</name>
<email>paul.burton@imgtec.com</email>
</author>
<published>2016-09-27T15:03:50Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=a920a17b2f418535870788ae81234dc6b8aa6661'/>
<id>urn:sha1:a920a17b2f418535870788ae81234dc6b8aa6661</id>
<content type='text'>
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 &lt;paul.burton@imgtec.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools: patman: Handle missing 'END' in non-last commit of a series</title>
<updated>2016-07-11T20:06:44Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng.cn@gmail.com</email>
</author>
<published>2016-06-27T06:24:32Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=94fbd3e37d6bdbf5490a185607ca20f862637220'/>
<id>urn:sha1:94fbd3e37d6bdbf5490a185607ca20f862637220</id>
<content type='text'>
The following python error:

Traceback (most recent call last):
  File "./tools/patman/patman", line 144, in &lt;module&gt;
    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 &lt;bmeng.cn@gmail.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools: patman: Handle missing blank line for 'Series-changes'</title>
<updated>2016-07-11T20:06:44Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng.cn@gmail.com</email>
</author>
<published>2016-06-27T06:24:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=57b6b190a8926403dca2444a45c34bed1f62992d'/>
<id>urn:sha1:57b6b190a8926403dca2444a45c34bed1f62992d</id>
<content type='text'>
'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 &lt;bmeng.cn@gmail.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools: patman: Generate cover letter correctly when 'END' is missing</title>
<updated>2016-07-11T20:06:44Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng.cn@gmail.com</email>
</author>
<published>2016-06-27T06:24:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=0d57718775243c2d2d7ff8c69dad83db08e1030d'/>
<id>urn:sha1:0d57718775243c2d2d7ff8c69dad83db08e1030d</id>
<content type='text'>
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 &lt;bmeng.cn@gmail.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools: patman: Handle tag sections without an 'END'</title>
<updated>2016-07-11T20:06:44Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng.cn@gmail.com</email>
</author>
<published>2016-06-27T06:24:29Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=13b98d95bab89bcac75c8a187577e7cc3754d194'/>
<id>urn:sha1:13b98d95bab89bcac75c8a187577e7cc3754d194</id>
<content type='text'>
'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 &lt;bmeng.cn@gmail.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>tools: patman: Use cover_match for 'Cover-letter'</title>
<updated>2016-07-11T20:06:44Z</updated>
<author>
<name>Bin Meng</name>
<email>bmeng.cn@gmail.com</email>
</author>
<published>2016-06-27T06:24:28Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=e7df218c3b446e02c5549b79dd76b65054d6147d'/>
<id>urn:sha1:e7df218c3b446e02c5549b79dd76b65054d6147d</id>
<content type='text'>
Like other patman tags, use a new variable cover_match to indicate
a match for 'Cover-letter'.

Signed-off-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Acked-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
</feed>
