summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKonstantin Ryabitsev <konstantin@linuxfoundation.org>2022-06-17 10:36:35 -0400
committerKonstantin Ryabitsev <konstantin@linuxfoundation.org>2022-06-17 10:36:35 -0400
commitb7ac0605e148a8d8808eeddd2e03e0ca6822cb5c (patch)
treee7a8b272a24845c5e39ab6f74f66029dc2cfff8d
parent797fc250c7dd64550f2283bdc749eda7555e3fa3 (diff)
downloadb4-b7ac0605e148a8d8808eeddd2e03e0ca6822cb5c.tar.gz
Add ability to exit before git-merge is invoked
It may be useful for the maintainer to review b4 retrieval/validation output before git-merge is invoked, so add a pause requiring an Enter or Ctrl-C. Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
-rw-r--r--b4/command.py2
-rw-r--r--b4/mbox.py10
-rw-r--r--man/b4.575
-rw-r--r--man/b4.5.rst4
4 files changed, 80 insertions, 11 deletions
diff --git a/b4/command.py b/b4/command.py
index 4f69231..92ffd65 100644
--- a/b4/command.py
+++ b/b4/command.py
@@ -117,6 +117,8 @@ def cmd():
help='Add more debugging info to the output')
parser.add_argument('-q', '--quiet', action='store_true', default=False,
help='Output critical information only')
+ parser.add_argument('-n', '--no-interactive', action='store_true', default=False,
+ help='Do not ask any interactive questions')
subparsers = parser.add_subparsers(help='sub-command help', dest='subcmd')
diff --git a/b4/mbox.py b/b4/mbox.py
index 277df89..1cc58c7 100644
--- a/b4/mbox.py
+++ b/b4/mbox.py
@@ -379,8 +379,16 @@ def make_am(msgs, cmdargs, msgid):
thanks_record_am(lser, cherrypick=cherrypick)
if cmdargs.merge:
+ if not cmdargs.no_interactive:
+ logger.info('Will exec: %s', ' '.join(mergecmd))
+ try:
+ input('Press Enter to continue or Ctrl-C to abort')
+ except KeyboardInterrupt:
+ logger.info('')
+ sys.exit(130)
+ else:
+ logger.info('Invoking: %s', ' '.join(mergecmd))
# We exec git-merge and let it take over
- logger.info('Invoking: %s', ' '.join(mergecmd))
os.execvp(mergecmd[0], mergecmd)
logger.info('You can now merge or checkout FETCH_HEAD')
diff --git a/man/b4.5 b/man/b4.5
index 150d193..86c5277 100644
--- a/man/b4.5
+++ b/man/b4.5
@@ -72,9 +72,17 @@ Add more debugging info to the output (default: False)
.TP
.B \-q\fP,\fB \-\-quiet
Output critical information only (default: False)
+.TP
+.B \-n\fP,\fB \-\-no\-interactive
+Do not ask any interactive questions (default: False)
.UNINDENT
.SH SUBCOMMAND OPTIONS
.SS b4 mbox
+.sp
+This command allows retrieving entire threads from a remote public\-inbox
+instance. The resulting mbox file can then be opened with most MUA
+clients for actions like replying to conversations or reviewing patch
+submissions.
.INDENT 0.0
.TP
.B usage:
@@ -117,6 +125,33 @@ When adding messages to existing maildir, filter out duplicates
.sp
\fIExample\fP: b4 mbox \fI\%20200313231252.64999\-1\-keescook@chromium.org\fP
.SS b4 am
+.sp
+This command allows retrieving threads from a public\-inbox instance and
+preparing them for applying to a git repository using the "git am"
+command. It will automatically perform the following operations:
+.INDENT 0.0
+.IP \(bu 2
+pick the latest submitted version of the series (it can check for
+newer threads using \fB\-c\fP as well)
+.IP \(bu 2
+check DKIM signatures and patatt attestation on all patches and code
+review messages
+.IP \(bu 2
+collate all submitted code\-review trailers (Reviewed\-by, Acked\-by,
+etc) and put them into the commit message
+.IP \(bu 2
+add your own Signed\-off\-by trailer (with \fB\-s\fP)
+.IP \(bu 2
+reroll series from partial updates (e.g. someone submits a v2 of a
+single patch instead of rerolling the entire series)
+.IP \(bu 2
+guess where in the tree history the patches belong, if the exact
+commit\-base is not specified (with \fB\-g\fP)
+.IP \(bu 2
+prepare the tree for a 3\-way merge (with \fB\-3\fP)
+.IP \(bu 2
+cherry\-pick a subset of patches from a large series (with \fB\-P\fP)
+.UNINDENT
.INDENT 0.0
.TP
.B usage:
@@ -210,11 +245,20 @@ Do not reroll partial series when detected
.UNINDENT
.sp
\fIExample\fP: b4 am \fI\%20200313231252.64999\-1\-keescook@chromium.org\fP
-.SH B4 SHAZAM
+.SS b4 shazam
+.sp
+This is very similar to \fBb4 am\fP, but will also apply patches
+directly to the current git tree using \fBgit am\fP\&. Alternatively, when
+used with \fB\-H\fP, it can fetch the patch series into \fBFETCH_HEAD\fP as
+if it were a pull request, so it can be reviewed and merged. In this
+case, the cover letter is used as a template for the merge commit.
+.sp
+If you want to automatically invoke git\-merge, you can use \fB\-M\fP
+instead of \fB\-H\fP\&.
.INDENT 0.0
.TP
.B usage:
-b4 shazam [\-h] [\-p USEPROJECT] [\-m LOCALMBOX] [\-C] [\-v WANTVER] [\-t] [\-S] [\-T] [\-s] [\-l] [\-P CHERRYPICK] [\-\-cc\-trailers] [\-\-no\-parent] [\-\-allow\-unicode\-control\-chars] [\-H] [\-\-guess\-lookback GUESSDAYS] [msgid]
+b4 shazam [\-h] [\-p USEPROJECT] [\-m LOCALMBOX] [\-C] [\-v WANTVER] [\-t] [\-S] [\-T] [\-s] [\-l] [\-P CHERRYPICK] [\-\-cc\-trailers] [\-\-no\-parent] [\-\-allow\-unicode\-control\-chars] [\-H | \-M] [\-\-guess\-lookback GUESSDAYS] [msgid]
.TP
.B positional arguments:
msgid Message ID to process, or pipe a raw message
@@ -267,16 +311,17 @@ Allow unicode control characters (very rarely legitimate)
.B \-H\fP,\fB \-\-make\-fetch\-head
Attempt to treat series as a pull request and fetch it into FETCH_HEAD
.TP
+.B \-M\fP,\fB \-\-merge
+Attempt to merge series as if it were a pull request (execs git\-merge)
+.TP
.BI \-\-guess\-lookback \ GUESSDAYS
-(use with \-H) When guessing base, go back this many days from the patch date (default: 3 weeks)
+(use with \-H or \-M) When guessing base, go back this many days from the patch date (default: 3 weeks)
.UNINDENT
.UNINDENT
.sp
\fIExample\fP: b4 shazam \-H \fI\%20200313231252.64999\-1\-keescook@chromium.org\fP
.SS b4 attest
.sp
-usage: b4 attest [\-h] patchfile [patchfile ...]
-.sp
\fBNOTE:\fP
.INDENT 0.0
.INDENT 3.5
@@ -290,12 +335,18 @@ same results.
.UNINDENT
.INDENT 0.0
.TP
+.B usage:
+b4 attest [\-h] patchfile [patchfile ...]
+.TP
.B positional arguments:
patchfile Patches to attest
.UNINDENT
.sp
\fIExample\fP: b4 attest outgoing/*.patch
.SS b4 pr
+.sp
+This command is for working with pull requests submitted using
+\fBgit\-request\-pull\fP\&.
.INDENT 0.0
.TP
.B usage:
@@ -389,10 +440,11 @@ files (global or in\-tree).
.sp
\fIExample\fP: b4 ty \-aS \-\-dry\-run
.SS b4 diff
-.sp
-usage: b4 diff [\-h] [\-g GITDIR] [\-p USEPROJECT] [\-C] [\-v WANTVERS [WANTVERS ...]] [\-n] [\-o OUTDIFF] [\-c] [\-m AMBOX AMBOX] [msgid]
.INDENT 0.0
.TP
+.B usage:
+b4 diff [\-h] [\-g GITDIR] [\-p USEPROJECT] [\-C] [\-v WANTVERS [WANTVERS ...]] [\-n] [\-o OUTDIFF] [\-c] [\-m AMBOX AMBOX] [msgid]
+.TP
.B positional arguments:
msgid Message ID to process, pipe a raw message, or use \-m
.UNINDENT
@@ -440,10 +492,11 @@ Compare two mbx files prepared with "b4 am"
.sp
\fIExample\fP: b4 diff \fI\%20200526205322.23465\-1\-mic@digikod.net\fP
.SS b4 kr
-.sp
-usage: b4 kr [\-h] [\-p USEPROJECT] [\-m LOCALMBOX] [\-C] [\-\-show\-keys] [msgid]
.INDENT 0.0
.TP
+.B usage:
+b4 kr [\-h] [\-p USEPROJECT] [\-m LOCALMBOX] [\-C] [\-\-show\-keys] [msgid]
+.TP
.B positional arguments:
msgid Message ID to process, or pipe a raw message
.TP
@@ -543,6 +596,10 @@ Default configuration, with explanations:
thanks\-pr\-template = None
# See thanks\-am\-template.example. If not set, a default template will be used.
thanks\-am\-template = None
+ # additional flags to pass to "git am" when we run "b4 shazam"
+ shazam\-am\-flags = None
+ # additional flags to pass to "git merge" when we run "b4 shazam \-M"
+ shazam\-merge\-flags = \-\-signoff
# Used when preparing merge messages from cover letters. See shazam\-merge\-template.example
shazam\-merge\-template = None
# Use to exclude certain mail addresses from ever being added to auto\-generated mail
diff --git a/man/b4.5.rst b/man/b4.5.rst
index 7bda7db..d4a26de 100644
--- a/man/b4.5.rst
+++ b/man/b4.5.rst
@@ -41,9 +41,11 @@ OPTIONS
-h, --help show this help message and exit
-d, --debug Add more debugging info to the output (default: False)
-q, --quiet Output critical information only (default: False)
+-n, --no-interactive Do not ask any interactive questions (default: False)
SUBCOMMAND OPTIONS
------------------
+
b4 mbox
~~~~~~~
@@ -163,7 +165,7 @@ options:
*Example*: b4 am 20200313231252.64999-1-keescook@chromium.org
b4 shazam
----------
+~~~~~~~~~
This is very similar to **b4 am**, but will also apply patches
directly to the current git tree using ``git am``. Alternatively, when