'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/patma
t for 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
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 pars
t for 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 +++
t
ython 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
t;patman -c1" 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 propagati
nd']
...but it doesn't because 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
comment about 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 checkp
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
t for 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
ython 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
t;patman -c1" 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(+),
nd']
...but it doesn't because 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 ch
aiming 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
T
yelling about "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 argum
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/pa
t for 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
ython 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
t;patman -c1" 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
nd']
...but it doesn't because 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
aiming 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
yelling about "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 w
hange-Id, which ends 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 (testBas
hange-Id, which ends 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-I
9-August/006739.html
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
25 matches
Mail list logo
