mirror of
https://github.com/AsahiLinux/u-boot
synced 2024-11-28 15:41:40 +00:00
patman: Add documentation to doc/
Link to patman's documentation from the doc/ directory so that it appears in the 'make htmldocs' output. Signed-off-by: Simon Glass <sjg@chromium.org> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
This commit is contained in:
Simon Glass
Heinrich Schuchardt
parent
9f0a2e77a0
commit
37c42b7270
6 changed files with 322 additions and 248 deletions
|
|
@ -14,6 +14,7 @@ General
|
|||
process
|
||||
release_cycle
|
||||
system_configuration
|
||||
sending_patches
|
||||
|
||||
Implementation
|
||||
--------------
|
||||
|
|
|
|||
1
doc/develop/patman.rst
Symbolic link
1
doc/develop/patman.rst
Symbolic link
|
|
@ -0,0 +1 @@
|
|||
../../tools/patman/patman.rst
|
||||
16
doc/develop/sending_patches.rst
Normal file
16
doc/develop/sending_patches.rst
Normal file
|
|
@ -0,0 +1,16 @@
|
|||
.. SPDX-License-Identifier: GPL-2.0+
|
||||
|
||||
Sending patches
|
||||
===============
|
||||
|
||||
.. toctree::
|
||||
:maxdepth: 2
|
||||
|
||||
patman
|
||||
|
||||
|
||||
You can use a tool called patman to prepare, check and sent patches. It creates
|
||||
change logs, cover letters and patch notes. It also simplified the process of
|
||||
sending multiple versions of a series.
|
||||
|
||||
See more details at :doc:`patman`.
|
||||
1
tools/patman/README.rst
Symbolic link
1
tools/patman/README.rst
Symbolic link
|
|
@ -0,0 +1 @@
|
|||
patman.rst
|
||||
|
|
@ -164,7 +164,8 @@ elif args.cmd == 'send':
|
|||
|
||||
elif args.full_help:
|
||||
tools.print_full_help(
|
||||
os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])), 'README')
|
||||
os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])),
|
||||
'README.rst')
|
||||
)
|
||||
|
||||
else:
|
||||
|
|
|
|||
|
|
@ -1,19 +1,31 @@
|
|||
# SPDX-License-Identifier: GPL-2.0+
|
||||
# Copyright (c) 2011 The Chromium OS Authors.
|
||||
.. SPDX-License-Identifier: GPL-2.0+
|
||||
.. Copyright (c) 2011 The Chromium OS Authors
|
||||
.. Simon Glass <sjg@chromium.org>
|
||||
.. v1, v2, 19-Oct-11
|
||||
.. revised v3 24-Nov-11
|
||||
.. revised v4 04-Jul-2020, with Patchwork integration
|
||||
|
||||
What is this?
|
||||
=============
|
||||
Patman patch manager
|
||||
====================
|
||||
|
||||
This tool is a Python script which:
|
||||
|
||||
- Creates patch directly from your branch
|
||||
|
||||
- Cleans them up by removing unwanted tags
|
||||
|
||||
- Inserts a cover letter with change lists
|
||||
|
||||
- Runs the patches through checkpatch.pl and its own checks
|
||||
|
||||
- Optionally emails them out to selected people
|
||||
|
||||
It also has some Patchwork features:
|
||||
|
||||
- shows review tags from Patchwork so you can update your local patches
|
||||
|
||||
- pulls these down into a new branch on request
|
||||
|
||||
- lists comments received on a series
|
||||
|
||||
It is intended to automate patch creation and make it a less
|
||||
|
|
@ -24,7 +36,7 @@ It is configured almost entirely by tags it finds in your commits.
|
|||
This means that you can work on a number of different branches at
|
||||
once, and keep the settings with each branch rather than having to
|
||||
git format-patch, git send-email, etc. with the correct parameters
|
||||
each time. So for example if you put:
|
||||
each time. So for example if you put::
|
||||
|
||||
Series-to: fred.blogs@napier.co.nz
|
||||
|
||||
|
|
@ -35,30 +47,33 @@ patches automatically (unless you use -m to disable this).
|
|||
|
||||
|
||||
How to use this tool
|
||||
====================
|
||||
--------------------
|
||||
|
||||
This tool requires a certain way of working:
|
||||
|
||||
- Maintain a number of branches, one for each patch series you are
|
||||
working on
|
||||
|
||||
- Add tags into the commits within each branch to indicate where the
|
||||
series should be sent, cover letter, version, etc. Most of these are
|
||||
normally in the top commit so it is easy to change them with 'git
|
||||
commit --amend'
|
||||
|
||||
- Each branch tracks the upstream branch, so that this script can
|
||||
automatically determine the number of commits in it (optional)
|
||||
|
||||
- Check out a branch, and run this script to create and send out your
|
||||
patches. Weeks later, change the patches and repeat, knowing that you
|
||||
will get a consistent result each time.
|
||||
|
||||
|
||||
How to configure it
|
||||
===================
|
||||
-------------------
|
||||
|
||||
For most cases of using patman for U-Boot development, patman can use the
|
||||
file 'doc/git-mailrc' in your U-Boot directory to supply the email aliases
|
||||
you need. To make this work, tell git where to find the file by typing
|
||||
this once:
|
||||
this once::
|
||||
|
||||
git config sendemail.aliasesfile doc/git-mailrc
|
||||
|
||||
|
|
@ -68,9 +83,8 @@ out where to send patches pretty well.
|
|||
During the first run patman creates a config file for you by taking the default
|
||||
user name and email address from the global .gitconfig file.
|
||||
|
||||
To add your own, create a file ~/.patman like this:
|
||||
To add your own, create a file ~/.patman like this::
|
||||
|
||||
>>>>
|
||||
# patman alias file
|
||||
|
||||
[alias]
|
||||
|
|
@ -80,8 +94,6 @@ u-boot: U-Boot Mailing List <u-boot@lists.denx.de>
|
|||
wolfgang: Wolfgang Denk <wd@denx.de>
|
||||
others: Mike Frysinger <vapier@gentoo.org>, Fred Bloggs <f.bloggs@napier.net>
|
||||
|
||||
<<<<
|
||||
|
||||
Aliases are recursive.
|
||||
|
||||
The checkpatch.pl in the U-Boot tools/ subdirectory will be located and
|
||||
|
|
@ -90,23 +102,17 @@ used. Failing that you can put it into your path or ~/bin/checkpatch.pl
|
|||
If you want to avoid sending patches to email addresses that are picked up
|
||||
by patman but are known to bounce you can add a [bounces] section to your
|
||||
.patman file. Unlike the [alias] section these are simple key: value pairs
|
||||
that are not recursive.
|
||||
|
||||
>>>
|
||||
that are not recursive::
|
||||
|
||||
[bounces]
|
||||
gonefishing: Fred Bloggs <f.bloggs@napier.net>
|
||||
|
||||
<<<
|
||||
|
||||
|
||||
If you want to change the defaults for patman's command-line arguments,
|
||||
you can add a [settings] section to your .patman file. This can be used
|
||||
for any command line option by referring to the "dest" for the option in
|
||||
patman.py. For reference, the useful ones (at the moment) shown below
|
||||
(all with the non-default setting):
|
||||
|
||||
>>>
|
||||
(all with the non-default setting)::
|
||||
|
||||
[settings]
|
||||
ignore_errors: True
|
||||
|
|
@ -115,56 +121,56 @@ verbose: True
|
|||
smtp_server: /path/to/sendmail
|
||||
patchwork_server: https://patchwork.ozlabs.org
|
||||
|
||||
<<<
|
||||
|
||||
|
||||
If you want to adjust settings (or aliases) that affect just a single
|
||||
project you can add a section that looks like [project_settings] or
|
||||
[project_alias]. If you want to use tags for your linux work, you could
|
||||
do:
|
||||
|
||||
>>>
|
||||
[project_alias]. If you want to use tags for your linux work, you could do::
|
||||
|
||||
[linux_settings]
|
||||
process_tags: True
|
||||
|
||||
<<<
|
||||
|
||||
|
||||
How to run it
|
||||
=============
|
||||
-------------
|
||||
|
||||
First do a dry run:
|
||||
|
||||
$ ./tools/patman/patman send -n
|
||||
.. code-block:: bash
|
||||
|
||||
./tools/patman/patman send -n
|
||||
|
||||
If it can't detect the upstream branch, try telling it how many patches
|
||||
there are in your series:
|
||||
there are in your series
|
||||
|
||||
$ ./tools/patman/patman -c5 send -n
|
||||
.. code-block:: bash
|
||||
|
||||
./tools/patman/patman -c5 send -n
|
||||
|
||||
This will create patch files in your current directory and tell you who
|
||||
it is thinking of sending them to. Take a look at the patch files.
|
||||
it is thinking of sending them to. Take a look at the patch files:
|
||||
|
||||
$ ./tools/patman/patman -c5 -s1 send -n
|
||||
.. code-block:: bash
|
||||
|
||||
./tools/patman/patman -c5 -s1 send -n
|
||||
|
||||
Similar to the above, but skip the first commit and take the next 5. This
|
||||
is useful if your top commit is for setting up testing.
|
||||
|
||||
|
||||
How to install it
|
||||
=================
|
||||
-----------------
|
||||
|
||||
The most up to date version of patman can be found in the U-Boot sources.
|
||||
However to use it on other projects it may be more convenient to install it as
|
||||
a standalone application. A distutils installer is included, this can be used
|
||||
to install patman:
|
||||
|
||||
$ cd tools/patman && python setup.py install
|
||||
.. code-block:: bash
|
||||
|
||||
cd tools/patman && python setup.py install
|
||||
|
||||
|
||||
How to add tags
|
||||
===============
|
||||
---------------
|
||||
|
||||
To make this script useful you must add tags like the following into any
|
||||
commit. Most can only appear once in the whole series.
|
||||
|
|
@ -206,6 +212,8 @@ Series-links: [id | version:id]...
|
|||
the series ID is 187331. This property can have a list of series IDs,
|
||||
one for each version of the series, e.g.
|
||||
|
||||
::
|
||||
|
||||
Series-links: 1:187331 2:188434 189372
|
||||
|
||||
Patman always uses the one without a version, since it assumes this is
|
||||
|
|
@ -219,35 +227,40 @@ Series-patchwork-url: url
|
|||
include the protocol and web site, with no trailing slash, for example
|
||||
'https://patchwork.ozlabs.org/project'
|
||||
|
||||
Cover-letter:
|
||||
Sets the cover letter contents for the series. The first line
|
||||
will become the subject of the cover letter::
|
||||
|
||||
Cover-letter:
|
||||
This is the patch set title
|
||||
blah blah
|
||||
more blah blah
|
||||
END
|
||||
Sets the cover letter contents for the series. The first line
|
||||
will become the subject of the cover letter
|
||||
|
||||
Cover-letter-cc: email / alias
|
||||
Additional email addresses / aliases to send cover letter to (you
|
||||
can add this multiple times)
|
||||
|
||||
Series-notes:
|
||||
Sets some notes for the patch series, which you don't want in
|
||||
the commit messages, but do want to send, The notes are joined
|
||||
together and put after the cover letter. Can appear multiple
|
||||
times::
|
||||
|
||||
Series-notes:
|
||||
blah blah
|
||||
blah blah
|
||||
more blah blah
|
||||
END
|
||||
Sets some notes for the patch series, which you don't want in
|
||||
the commit messages, but do want to send, The notes are joined
|
||||
together and put after the cover letter. Can appear multiple
|
||||
times.
|
||||
|
||||
Commit-notes:
|
||||
Similar, but for a single commit (patch). These notes will appear
|
||||
immediately below the --- cut in the patch file::
|
||||
|
||||
Commit-notes:
|
||||
blah blah
|
||||
blah blah
|
||||
more blah blah
|
||||
END
|
||||
Similar, but for a single commit (patch). These notes will appear
|
||||
immediately below the --- cut in the patch file.
|
||||
|
||||
Signed-off-by: Their Name <email>
|
||||
A sign-off is added automatically to your patches (this is
|
||||
|
|
@ -255,19 +268,20 @@ END
|
|||
override the default signoff that patman automatically adds.
|
||||
Multiple duplicate signoffs will be removed.
|
||||
|
||||
Tested-by: Their Name <email>
|
||||
Reviewed-by: Their Name <email>
|
||||
Acked-by: Their Name <email>
|
||||
Tested-by / Reviewed-by / Acked-by
|
||||
These indicate that someone has tested/reviewed/acked your patch.
|
||||
When you get this reply on the mailing list, you can add this
|
||||
tag to the relevant commit and the script will include it when
|
||||
you send out the next version. If 'Tested-by:' is set to
|
||||
yourself, it will be removed. No one will believe you.
|
||||
|
||||
Example::
|
||||
|
||||
Tested-by: Their Name <fred@bloggs.com>
|
||||
Reviewed-by: Their Name <email>
|
||||
Acked-by: Their Name <email>
|
||||
|
||||
Series-changes: n
|
||||
- Guinea pig moved into its cage
|
||||
- Other changes ending with a blank line
|
||||
<blank line>
|
||||
This can appear in any commit. It lists the changes for a
|
||||
particular version n of that commit. The change list is
|
||||
created based on this information. Each commit gets its own
|
||||
|
|
@ -279,23 +293,38 @@ Series-changes: n
|
|||
to update the log there and then, knowing that the script will
|
||||
do the rest.
|
||||
|
||||
Commit-changes: n
|
||||
- This line will not appear in the cover-letter changelog
|
||||
Example::
|
||||
|
||||
Series-changes: n
|
||||
- Guinea pig moved into its cage
|
||||
- Other changes ending with a blank line
|
||||
<blank line>
|
||||
|
||||
Commit-changes: n
|
||||
This tag is like Series-changes, except changes in this changelog will
|
||||
only appear in the changelog of the commit this tag is in. This is
|
||||
useful when you want to add notes which may not make sense in the cover
|
||||
letter. For example, you can have short changes such as "New" or
|
||||
"Lint".
|
||||
|
||||
Cover-changes: n
|
||||
- This line will only appear in the cover letter
|
||||
Example::
|
||||
|
||||
Commit-changes: n
|
||||
- This line will not appear in the cover-letter changelog
|
||||
<blank line>
|
||||
|
||||
Cover-changes: n
|
||||
This tag is like Series-changes, except changes in this changelog will
|
||||
only appear in the cover-letter changelog. This is useful to summarize
|
||||
changes made with Commit-changes, or to add additional context to
|
||||
changes.
|
||||
|
||||
Example::
|
||||
|
||||
Cover-changes: n
|
||||
- This line will only appear in the cover letter
|
||||
<blank line>
|
||||
|
||||
Patch-cc: Their Name <email>
|
||||
This copies a single patch to another email address. Note that the
|
||||
Cc: used by git send-email is ignored by patman, but will be
|
||||
|
|
@ -306,6 +335,8 @@ Series-process-log: sort, uniq
|
|||
multiple lines long, as long as each subsequent line of a change begins
|
||||
with a whitespace character. For example,
|
||||
|
||||
Example::
|
||||
|
||||
- This change
|
||||
continues onto the next line
|
||||
- But this change is separate
|
||||
|
|
@ -322,7 +353,7 @@ Change-Id:
|
|||
sent out with the same Change-Id.
|
||||
|
||||
Various other tags are silently removed, like these Chrome OS and
|
||||
Gerrit tags:
|
||||
Gerrit tags::
|
||||
|
||||
BUG=...
|
||||
TEST=...
|
||||
|
|
@ -335,15 +366,14 @@ patch series and see how the patches turn out.
|
|||
|
||||
|
||||
Where Patches Are Sent
|
||||
======================
|
||||
----------------------
|
||||
|
||||
Once the patches are created, patman sends them using git send-email. The
|
||||
whole series is sent to the recipients in Series-to: and Series-cc.
|
||||
You can Cc individual patches to other people with the Patch-cc: tag. Tags
|
||||
in the subject are also picked up to Cc patches. For example, a commit like
|
||||
this:
|
||||
this::
|
||||
|
||||
>>>>
|
||||
commit 10212537b85ff9b6e09c82045127522c0f0db981
|
||||
Author: Mike Frysinger <vapier@gentoo.org>
|
||||
Date: Mon Nov 7 23:18:44 2011 -0500
|
||||
|
|
@ -354,14 +384,13 @@ Date: Mon Nov 7 23:18:44 2011 -0500
|
|||
|
||||
Patch-cc: sandbox, mikef, ag
|
||||
Patch-cc: afleming
|
||||
<<<<
|
||||
|
||||
will create a patch which is copied to x86, arm, sandbox, mikef, ag and
|
||||
afleming.
|
||||
|
||||
If you have a cover letter it will get sent to the union of the Patch-cc
|
||||
lists of all of the other patches. If you want to sent it to additional
|
||||
people you can add a tag:
|
||||
people you can add a tag::
|
||||
|
||||
Cover-letter-cc: <list of addresses>
|
||||
|
||||
|
|
@ -370,21 +399,23 @@ list for any of the patches.
|
|||
|
||||
|
||||
Patchwork Integration
|
||||
=====================
|
||||
---------------------
|
||||
|
||||
Patman has a very basic integration with Patchwork. If you point patman to
|
||||
your series on patchwork it can show you what new reviews have appears since
|
||||
your series on patchwork it can show you what new reviews have appeared since
|
||||
you sent your series.
|
||||
|
||||
To set this up, add a Series-link tag to one of the commits in your series
|
||||
(see above).
|
||||
|
||||
Then you can type
|
||||
Then you can type:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman status
|
||||
|
||||
and patman will show you each patch and what review tags have been collected,
|
||||
for example:
|
||||
for example::
|
||||
|
||||
...
|
||||
21 x86: mtrr: Update the command to use the new mtrr
|
||||
|
|
@ -402,6 +433,8 @@ series.
|
|||
|
||||
To automatically pull into these tags into a new branch, use the -d option:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman status -d mtrr4
|
||||
|
||||
This will create a new 'mtrr4' branch which is the same as your current branch
|
||||
|
|
@ -409,6 +442,8 @@ but has the new review tags in it. The tags are added in alphabetic order and
|
|||
are placed immediately after any existing ack/review/test/fixes tags, or at the
|
||||
end. You can check that this worked with:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman -b mtrr4 status
|
||||
|
||||
which should show that there are no new responses compared to this new branch.
|
||||
|
|
@ -417,7 +452,7 @@ There is also a -C option to list the comments received for each patch.
|
|||
|
||||
|
||||
Example Work Flow
|
||||
=================
|
||||
-----------------
|
||||
|
||||
The basic workflow is to create your commits, add some tags to the top
|
||||
commit, and type 'patman' to check and send them.
|
||||
|
|
@ -425,7 +460,7 @@ commit, and type 'patman' to check and send them.
|
|||
Here is an example workflow for a series of 4 patches. Let's say you have
|
||||
these rather contrived patches in the following order in branch us-cmd in
|
||||
your tree where 'us' means your upstreaming activity (newest to oldest as
|
||||
output by git log --oneline):
|
||||
output by git log --oneline)::
|
||||
|
||||
7c7909c wip
|
||||
89234f5 Don't include standard parser if hush is used
|
||||
|
|
@ -438,36 +473,46 @@ but that you don't want to submit because there is an existing patch for it
|
|||
on the list. So you can tell patman to create and check some patches
|
||||
(skipping the first patch) with:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman -s1 send -n
|
||||
|
||||
If you want to do all of them including the work-in-progress one, then
|
||||
(if you are tracking an upstream branch):
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman send -n
|
||||
|
||||
Let's say that patman reports an error in the second patch. Then:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
git rebase -i HEAD~6
|
||||
<change 'pick' to 'edit' in 89234f5>
|
||||
<use editor to make code changes>
|
||||
# change 'pick' to 'edit' in 89234f5
|
||||
# use editor to make code changes
|
||||
git add -u
|
||||
git rebase --continue
|
||||
|
||||
Now you have an updated patch series. To check it:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman -s1 send -n
|
||||
|
||||
Let's say it is now clean and you want to send it. Now you need to set up
|
||||
the destination. So amend the top commit with:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
git commit --amend
|
||||
|
||||
Use your editor to add some tags, so that the whole commit message is:
|
||||
Use your editor to add some tags, so that the whole commit message is::
|
||||
|
||||
The current run_command() is really only one of the options, with
|
||||
hush providing the other. It really shouldn't be called directly
|
||||
in case the hush parser is bring used, so rename this function to
|
||||
better explain its purpose.
|
||||
better explain its purpose::
|
||||
|
||||
Series-to: u-boot
|
||||
Series-cc: bfin, marex
|
||||
|
|
@ -490,6 +535,8 @@ mmc and sparc, and the last one to sandbox.
|
|||
|
||||
Now to send the patches, take off the -n flag:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman -s1 send
|
||||
|
||||
The patches will be created, shown in your editor, and then sent along with
|
||||
|
|
@ -502,36 +549,42 @@ Also, the patch on the list that you were waiting for has been merged,
|
|||
so you can drop your wip commit.
|
||||
|
||||
Take a look on patchwork and find out the URL of the series. This will be
|
||||
something like http://patchwork.ozlabs.org/project/uboot/list/?series=187331
|
||||
Add this to a tag in your top commit:
|
||||
something like `http://patchwork.ozlabs.org/project/uboot/list/?series=187331`
|
||||
Add this to a tag in your top commit::
|
||||
|
||||
Series-link: http://patchwork.ozlabs.org/project/uboot/list/?series=187331
|
||||
Series-links: 187331
|
||||
|
||||
You can use then patman to collect the Acked-by tag to the correct commit,
|
||||
creating a new 'version 2' branch for us-cmd:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman status -d us-cmd2
|
||||
git checkout us-cmd2
|
||||
|
||||
You can look at the comments in Patchwork or with:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman status -C
|
||||
|
||||
Then you can resync with upstream:
|
||||
|
||||
git fetch origin (or whatever upstream is called)
|
||||
.. code-block:: bash
|
||||
|
||||
git fetch origin # or whatever upstream is called
|
||||
git rebase origin/master
|
||||
|
||||
and use git rebase -i to edit the commits, dropping the wip one.
|
||||
|
||||
Then update the Series-cc: in the top commit to add the person who reviewed
|
||||
the v1 series:
|
||||
Then update the `Series-cc:` in the top commit to add the person who reviewed
|
||||
the v1 series::
|
||||
|
||||
Series-cc: bfin, marex, Heiko Schocher <hs@denx.de>
|
||||
|
||||
and remove the Series-prefix: tag since it it isn't an RFC any more. The
|
||||
series is now version two, so the series info in the top commit looks like
|
||||
this:
|
||||
this::
|
||||
|
||||
Series-to: u-boot
|
||||
Series-cc: bfin, marex, Heiko Schocher <hs@denx.de>
|
||||
|
|
@ -541,7 +594,7 @@ this:
|
|||
|
||||
Finally, you need to add a change log to the two commits you changed. You
|
||||
add change logs to each individual commit where the changes happened, like
|
||||
this:
|
||||
this::
|
||||
|
||||
Series-changes: 2
|
||||
- Updated the command decoder to reduce code size
|
||||
|
|
@ -551,7 +604,7 @@ this:
|
|||
|
||||
When you run patman it will collect all the change logs from the different
|
||||
commits and combine them into the cover letter, if you have one. So finally
|
||||
you have a new series of commits:
|
||||
you have a new series of commits::
|
||||
|
||||
faeb973 Don't include standard parser if hush is used
|
||||
1b2f2fe mmc: sparc: Stop using builtin_run_command()
|
||||
|
|
@ -560,13 +613,15 @@ you have a new series of commits:
|
|||
|
||||
so to send them:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
patman
|
||||
|
||||
and it will create and send the version 2 series.
|
||||
|
||||
|
||||
General points
|
||||
==============
|
||||
--------------
|
||||
|
||||
1. When you change back to the us-cmd branch days or weeks later all your
|
||||
information is still there, safely stored in the commits. You don't need
|
||||
|
|
@ -580,8 +635,10 @@ automatically in many cases.
|
|||
compare change and see what you did, you can either create a new branch for
|
||||
each version, or just tag the branch before you start changing it:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
git tag sent/us-cmd-rfc
|
||||
...later...
|
||||
# ...later...
|
||||
git tag sent/us-cmd-v2
|
||||
|
||||
4. If you want to modify the patches a little before sending, you can do
|
||||
|
|
@ -603,7 +660,7 @@ a link to a git repository in your "Commit-notes", "Series-notes" or
|
|||
8. Patches will have no changelog entries for revisions where they did not
|
||||
change. For clarity, if there are no changes for this patch in the most
|
||||
recent revision of the series, a note will be added. For example, a patch
|
||||
with the following tags in the commit
|
||||
with the following tags in the commit::
|
||||
|
||||
Series-version: 5
|
||||
Series-changes: 2
|
||||
|
|
@ -612,7 +669,7 @@ with the following tags in the commit
|
|||
Series-changes: 4
|
||||
- Another change
|
||||
|
||||
would have a changelog of
|
||||
would have a changelog of:::
|
||||
|
||||
(no changes since v4)
|
||||
|
||||
|
|
@ -622,8 +679,9 @@ would have a changelog of
|
|||
Changes in v2:
|
||||
- Some change
|
||||
|
||||
|
||||
Other thoughts
|
||||
==============
|
||||
--------------
|
||||
|
||||
This script has been split into sensible files but still needs work.
|
||||
Most of these are indicated by a TODO in the code.
|
||||
|
|
@ -633,6 +691,8 @@ It would be nice if this could handle the In-reply-to side of things.
|
|||
The tests are incomplete, as is customary. Use the 'test' subcommand to run
|
||||
them:
|
||||
|
||||
.. code-block:: bash
|
||||
|
||||
$ tools/patman/patman test
|
||||
|
||||
Error handling doesn't always produce friendly error messages - e.g.
|
||||
|
|
@ -641,9 +701,3 @@ putting an incorrect tag in a commit may provide a confusing message.
|
|||
There might be a few other features not mentioned in this README. They
|
||||
might be bugs. In particular, tags are case sensitive which is probably
|
||||
a bad thing.
|
||||
|
||||
|
||||
Simon Glass <sjg@chromium.org>
|
||||
v1, v2, 19-Oct-11
|
||||
revised v3 24-Nov-11
|
||||
revised v4 Independence Day 2020, with Patchwork integration
|
||||
Loading…
Reference in a new issue