[U-Boot] [PATCH v1] patman: add Series-commit-notes tag and section

Sometimes a commit should have notes enclosed with it rather than withing the cover letter -- possibly even because there is no cover letter. Add 'Series-commit-notes' tag, similar to 'Series-notes'; lines between this tag and the next END line are inserted in the patch right after the '---' separator line.
Signed-off-by: Albert ARIBAUD albert.u.boot@aribaud.net ---
tools/patman/commit.py | 2 ++ tools/patman/patchstream.py | 7 +++++-- tools/patman/series.py | 2 +- 3 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/tools/patman/commit.py b/tools/patman/commit.py index 900cfb3..89cce7f 100644 --- a/tools/patman/commit.py +++ b/tools/patman/commit.py @@ -21,6 +21,7 @@ class Commit: changes: Dict containing a list of changes (single line strings). The dict is indexed by change version (an integer) cc_list: List of people to aliases/emails to cc on this commit + notes: List of lines in the commit (not series) notes """ def __init__(self, hash): self.hash = hash @@ -28,6 +29,7 @@ class Commit: self.tags = [] self.changes = {} self.cc_list = [] + self.notes = []
def AddChange(self, version, info): """Add a new change line to the change list for a version. diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py index c204523..4c20144 100644 --- a/tools/patman/patchstream.py +++ b/tools/patman/patchstream.py @@ -84,7 +84,7 @@ class PatchStream: name: Tag name (part after 'Series-') value: Tag value (part after 'Series-xxx: ') """ - if name == 'notes': + if (name == 'notes' or name == 'commit-notes'): self.in_section = name self.skip_blank = False if self.is_log: @@ -165,6 +165,9 @@ class PatchStream: elif self.in_section == 'notes': if self.is_log: self.series.notes += self.section + elif self.in_section == 'commit-notes': + if self.is_log: + self.commit.notes += self.section else: self.warn.append("Unknown section '%s'" % self.in_section) self.in_section = None @@ -276,7 +279,7 @@ class PatchStream: out = [] log = self.series.MakeChangeLog(self.commit) out += self.FormatTags(self.tags) - out += [line] + log + out += [line] + self.commit.notes + [''] + log elif self.found_test: if not re_allowed_after_test.match(line): self.lines_after_test += 1 diff --git a/tools/patman/series.py b/tools/patman/series.py index 88c0d87..5eeb452 100644 --- a/tools/patman/series.py +++ b/tools/patman/series.py @@ -12,7 +12,7 @@ import terminal
# Series-xxx tags that we understand valid_series = ['to', 'cc', 'version', 'changes', 'prefix', 'notes', 'name', - 'cover-cc', 'process_log'] + 'cover-cc', 'process_log', 'commit_notes']
class Series(dict): """Holds information about a patch series, including all tags.

Hello Albert, Simon.
Sometimes a commit should have notes enclosed with it rather than withing the cover letter -- possibly even because there is no cover letter. Add 'Series-commit-notes' tag, similar to 'Series-notes'; lines between this tag and the next END line are inserted in the patch right after the '---' separator line.
(Please correct me if I am wrong.)
In my understanding, "Series-" prefix implies that the tag has an impact on the whole series. For example, Series-to, Series-cc, Series-version, Series-prefix, etc.
If I am not misunderstanding, in this case 'Series-commit-notes' tag adds a ratinale part to the only patch file which this tag is given to.
How about dropping 'Series-' prefix?
Best Regards Masahiro Yamada

Hi Massihiro,
On Mon, Nov 11, 2013 at 6:22 PM, Masahiro Yamada yamada.m@jp.panasonic.comwrote:
Hello Albert, Simon.
Sometimes a commit should have notes enclosed with it rather than withing the cover letter -- possibly even because there is no cover letter. Add 'Series-commit-notes' tag, similar to 'Series-notes'; lines between this tag and the next END line are inserted in the patch right after the '---' separator line.
(Please correct me if I am wrong.)
In my understanding, "Series-" prefix implies that the tag has an impact on the whole series. For example, Series-to, Series-cc, Series-version, Series-prefix, etc.
If I am not misunderstanding, in this case 'Series-commit-notes' tag adds a ratinale part to the only patch file which this tag is given to.
How about dropping 'Series-' prefix?
Yes I think that is a good idea.
Regards, Simon
participants (3)
-
Albert ARIBAUD
-
Masahiro Yamada
-
Simon Glass