Simon,
On Wed, Mar 20, 2013 at 7:43 PM, Simon Glass <s...@chromium.org> wrote:
> For some series with lots of changes it is annoying that duplicate change
> log items are not caught. It is also helpful sometimes to sort the change
> logs.
>
> Add a Series-process-log tag to enable this, which can be placed in a
> commit to control this.
>
> The change to the Cc: line is to fix a checkpatch warning.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
> tools/patman/README | 8 +++++++-
> tools/patman/patchstream.py | 2 +-
> tools/patman/series.py | 8 ++++++--
> 3 files changed, 14 insertions(+), 4 deletions(-)
Not sure I'd find this terribly useful myself, but I don't see
anything wrong with it. I think my change log items tend to be more
than one line long for one...
> + if not ('uniq' in process_it and text in out):
optional: My brain had a hard time processing this. I did the logic
transformation myself:
if 'uniq' not in process_it or text not in out:
Also: Do you really want the "process_it" to be so free-form? That
seems like it might be asking for disaster. Why not specify that it's
comma-separated and be done.
-Doug
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
