diff options
| author | Simon Glass <sjg@chromium.org> | 2014-09-05 19:00:19 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2014-09-09 16:38:30 -0600 |
| commit | 891b7a076146db7984388e5b8388101eb4f44347 (patch) | |
| tree | 950ac39f3a54fb398df251402281d37f754ed85e /tools | |
| parent | 883a321a4bee2d2b2f16e445060c17494e76bc48 (diff) | |
| download | blackbird-obmc-uboot-891b7a076146db7984388e5b8388101eb4f44347.tar.gz blackbird-obmc-uboot-891b7a076146db7984388e5b8388101eb4f44347.zip | |
patman: Start with a clean series when needed
For reasons that are not well-understood, GetMetaDataForList() can end up
adding to an existing series even when it appears that it should be
starting a new one.
Change from using a default constructor parameter to an explicit one, to
work around this problem.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/patman/patchstream.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py index b0b81534bf..b3e66c32a9 100644 --- a/tools/patman/patchstream.py +++ b/tools/patman/patchstream.py @@ -355,7 +355,7 @@ class PatchStream: def GetMetaDataForList(commit_range, git_dir=None, count=None, - series = Series()): + series = None): """Reads out patch series metadata from the commits This does a 'git log' on the relevant commits and pulls out the tags we @@ -370,6 +370,8 @@ def GetMetaDataForList(commit_range, git_dir=None, count=None, Returns: A Series object containing information about the commits. """ + if not series: + series = Series() params = gitutil.LogCmd(commit_range,reverse=True, count=count, git_dir=git_dir) stdout = command.RunPipe([params], capture=True).stdout |
