[PATCH 3/3] patman: Add a tag for when a patch gets added to a series

Simon Glass sjg at chromium.org
Mon Jul 15 15:31:56 CEST 2024


On Mon, 29 Apr 2024 at 17:49, Doug Anderson <dianders at chromium.org> wrote:
>
> Hi,
>
> On Thu, Apr 18, 2024 at 7:36 PM Sean Anderson <seanga2 at gmail.com> wrote:
> >
> > When a patch is added to a series after the initial version, there are no
> > changes to note except that it is new. This is typically done to suppress
> > the "(no changes in vN)" message. It's also nice to add a change to the
> > cover letter so reviewers know there is an additional patch. Add a tag to
> > automate this process a bit.
> >
> > There are two nits with the current approach:
> >
> > - It favors '-' as a bullet point, but some people may prefer '*' (or
> >   something else)
> > - Tags (e.g. 'patman: ' in 'patman: foo bar') are not stripped. They are
> >   probably just noise in most series, but they may be useful for treewide
> >   series to distinguish 'gpio: frobnicate' from 'reset: frobnicate', so
> >   I've left them in.
> >
> > Suggestions for the above appreciated.
> >
> > Suggested-by: Douglas Anderson <dianders at chromium.org>
> > Signed-off-by: Sean Anderson <seanga2 at gmail.com>
> > ---
> >
> >  tools/patman/func_test.py                           |  2 ++
> >  tools/patman/patchstream.py                         |  5 +++++
> >  tools/patman/patman.rst                             | 13 +++++++++++++
> >  ...t-cast-for-sandbox-in-fdtdec_setup_mem_siz.patch |  1 +
> >  tools/patman/test/test01.txt                        |  1 +
> >  5 files changed, 22 insertions(+)
>
> I love it, thanks!
>
> Reviewed-by: Douglas Anderson <dianders at chromium.org>

Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list