[PATCH] patman: Properly document the patchwork_url setting

2024-03-11 Thread Douglas Anderson
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

[PATCH v4] patman: By default don't pass "--no-tree" to checkpatch for linux

2022-07-19 Thread Douglas Anderson
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

[PATCH v3 5/5] patman: Take project defaults into account for --help

2022-07-07 Thread Douglas Anderson
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

[PATCH v3 4/5] patman: By default don't pass "--no-tree" to checkpatch for linux

2022-07-07 Thread Douglas Anderson
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

[PATCH v3 3/5] patman: Make most bool arguments BooleanOptionalAction

2022-07-07 Thread Douglas Anderson
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

[PATCH v3 2/5] patman: Don't look at sys.argv when parsing settings

2022-07-07 Thread Douglas Anderson
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

[PATCH v3 1/5] patman: Fix implicit command inserting

2022-07-07 Thread Douglas Anderson
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

[PATCH v3 0/5] patman: Small fixes plus remove --no-tree from checkpatch for linux

2022-07-07 Thread Douglas Anderson
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

[PATCH v2 6/6] patman: Take project defaults into account for --help

2022-07-01 Thread Douglas Anderson
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

[PATCH v2 5/6] patman: By default don't pass "--no-tree" to checkpatch for linux

2022-07-01 Thread Douglas Anderson
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 ---

[PATCH v2 4/6] patman: Make most bool arguments BooleanOptionalAction

2022-07-01 Thread Douglas Anderson
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

[PATCH v2 3/6] patman: Don't look at sys.argv when parsing settings

2022-07-01 Thread Douglas Anderson
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

[PATCH v2 2/6] patman: Fix implicit command inserting

2022-07-01 Thread Douglas Anderson
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

[PATCH v2 1/6] patman: Fix updating argument defaults from settings

2022-07-01 Thread Douglas Anderson
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

[PATCH v2 0/6] patman: Small fixes plus remove --no-tree from checkpatch for linux

2022-07-01 Thread Douglas Anderson
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

[PATCH 6/6] patman: Take project defaults into account for --help

2022-06-30 Thread Douglas Anderson
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.

[PATCH 5/6] patman: By default don't pass "--no-tree" to checkpatch for linux

2022-06-30 Thread Douglas Anderson
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 ++-

[PATCH 4/6] patman: Make most bool arguments BooleanOptionalAction

2022-06-30 Thread Douglas Anderson
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

[PATCH 3/6] patman: Don't look at sys.argv when parsing settings

2022-06-30 Thread Douglas Anderson
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.

[PATCH 2/6] patman: Fix implicit command inserting

2022-06-30 Thread Douglas Anderson
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

[PATCH 1/6] patman: Fix updating argument defaults from settings

2022-06-30 Thread Douglas Anderson
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

[PATCH 0/6] patman: Small fixes plus remove --no-tree from checkpatch for linux

2022-06-30 Thread Douglas Anderson
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

[U-Boot] [PATCH v3] patman: Use the Change-Id, version, and prefix in the Message-Id

2019-09-27 Thread Douglas Anderson
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

[U-Boot] [PATCH v2] patman: Use the Change-Id, version, and prefix in the Message-Id

2019-09-03 Thread Douglas Anderson
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.

[U-Boot] [RFC PATCH] patman: Use the Change-Id, version, and prefix in the Message-Id

2019-08-28 Thread Douglas Anderson
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