s actually not a valid
setting. It should be "patchwork_url".
Fix these problems so the doc is right and more useful.
Signed-off-by: Douglas Anderson
---
tools/patman/patman.rst | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/patman/patman.rst b/to
Linux to _not_ pass --no-tree. We'll have a
config option and command line flag to override.
Signed-off-by: Douglas Anderson
---
Without BooleanOptionalAction (python 3.9), it's a little odd to make
the help text understandable while having the help text account for
the project settings when
I'd like it so that when you do "patman send --help" and you're using
Linux that it show it the proper defaults for Linux.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Simon Glass
---
(no changes since v2)
Changes in v2:
- Make comment about parsing three
Linux to _not_ pass --no-tree. We'll have a
config option and command line flag to override.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
Reviewed-by: Simon Glass
---
(no changes since v1)
tools/patman/checkpatch.py | 11 +++
tools/patma
handles this well--we just need to specify things with
BooleanOptionalAction. We'll do that for all options except
"full-help" (where it feels silly). This uglifies the help text a
little bit but does give maximum flexibility.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
ot; if you don't have the patch ("patman: Fix implicit
command inserting") before it.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
Reviewed-by: Simon Glass
---
Changes in v3:
- Rebased atop ("patman: Fix defaults not propagating to subparsers&quo
argv[:-0] is the same as argv[:0] and that's
an empty list.
Let's fix this little glitch.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
Reviewed-by: Simon Glass
---
(no changes since v1)
tools/patman/main.py | 4 +++-
1 file changed, 3 insertions(+), 1 d
out parsing three times less nonsensical.
Douglas Anderson (5):
patman: Fix implicit command inserting
patman: Don't look at sys.argv when parsing settings
patman: Make most bool arguments BooleanOptionalAction
patman: By default don't pass "--no-tree" to checkpatch for linux
pat
I'd like it so that when you do "patman send --help" and you're using
Linux that it show it the proper defaults for Linux.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
---
Changes in v2:
- Make comment about parsing three times less nonsensical.
tools/patman/ma
Linux to _not_ pass --no-tree. We'll have a
config option and command line flag to override.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
---
(no changes since v1)
tools/patman/checkpatch.py | 11 +++
tools/patman/control.py| 7 ---
handles this well--we just need to specify things with
BooleanOptionalAction. We'll do that for all options except
"full-help" (where it feels silly). This uglifies the help text a
little bit but does give maximum flexibility.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
ot; if you don't have the patch ("patman: Fix implicit
command inserting") before it.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
---
(no changes since v1)
tools/patman/settings.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
argv[:-0] is the same as argv[:0] and that's
an empty list.
Let's fix this little glitch.
Signed-off-by: Douglas Anderson
Tested-by: Brian Norris
Reviewed-by: Brian Norris
---
(no changes since v1)
tools/patman/main.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
t claiming that the defaults propagates
through all subparsers no longer works for some reason.
We're already looking at all the subparsers anyway. Let's just update
each one.
Fixes: 4600767d294d ("patman: Refactor how the default subcommand works")
Signed-off-by: Douglas Anderson
Tested-by: Br
bout "tags" when used with Linux even though Linux is
supposed to have "process_tags" defaulted to False.
Changes in v2:
- Fix doc string for --ignore-bad-tags
- Make comment about parsing three times less nonsensical.
Douglas Anderson (6):
patman: Fix updating argument defaul
I'd like it so that when you do "patman send --help" and you're using
Linux that it show it the proper defaults for Linux.
Signed-off-by: Douglas Anderson
---
tools/patman/main.py | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/tools/patman/main.
Linux to _not_ pass --no-tree. We'll have a
config option and command line flag to override.
Signed-off-by: Douglas Anderson
---
tools/patman/checkpatch.py | 11 +++
tools/patman/control.py| 7 ---
tools/patman/main.py | 3 +++
tools/patman/settings.py | 3 ++-
handles this well--we just need to specify things with
BooleanOptionalAction. We'll do that for all options except
"full-help" (where it feels silly). This uglifies the help text a
little bit but does give maximum flexibility.
Signed-off-by: Douglas Anderson
---
tools/patman/ma
ot; if you don't have the patch ("patman: Fix implicit
command inserting") before it.
Signed-off-by: Douglas Anderson
---
tools/patman/settings.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/patman/settings.py b/tools/patman/settings.py
index 5eefe3d1f55e.
argv[:-0] is the same as argv[:0] and that's
an empty list.
Let's fix this little glitch.
Signed-off-by: Douglas Anderson
---
tools/patman/main.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/patman/main.py b/tools/patman/main.py
index 2a2ac4570931..336f4e43
t claiming that the defaults propagates
through all subparsers no longer works for some reason.
We're already looking at all the subparsers anyway. Let's just update
each one.
Fixes: 4600767d294d ("patman: Refactor how the default subcommand works")
Signed-off-by: Douglas Anderson
---
tools/pat
bout "tags" when used with Linux even though Linux is
supposed to have "process_tags" defaulted to False.
Douglas Anderson (6):
patman: Fix updating argument defaults from settings
patman: Fix implicit command inserting
patman: Don't look at sys.argv when parsing set
nds in 2b9 if you want to
check.
[1]
https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2019-August/006739.html
Signed-off-by: Douglas Anderson
---
Changes in v3:
- Allow faking the time of Message-Id for testing (testBasic)
- Add skip_blank for Change-Id (testBasic).
- Check the
nds in 2b9 if you want to
check.
[1]
https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2019-August/006739.html
Signed-off-by: Douglas Anderson
---
Changes in v2:
- Add a "v" before the version part of the Message-Id
- Reorder the parts of the Message-Id as per Johannes.
Signed-off-by: Douglas Anderson
---
tools/patman/README | 8 +-
tools/patman/commit.py | 3 ++
tools/patman/patchstream.py | 56 +++--
3 files changed, 64 insertions(+), 3 deletions(-)
diff --git a/tools/patman/README b/tools/patman/README
25 matches
Mail list logo