[v2] scripts: render_block_graph: Fix with new python and improve argument parsing

[PATCH v2 0/3] scripts: render_block_graph: Fix with new python and improve argument parsing

Peter Krempa posted 3 patches 1 week, 6 days ago

Diff against v1
Download series mbox

scripts/render_block_graph.py | 80 +++++++++++++++++++++++++----------
1 file changed, 57 insertions(+), 23 deletions(-)

Expand all Fold all

[PATCH v2 0/3] scripts: render_block_graph: Fix with new python and improve argument parsing

Posted by Peter Krempa 1 week, 6 days ago

New python doesn't like '\l' escapes from the 'dot' language.

While at it improve usability of the script by employing proper argument
parsing.

v2:
 - use mutually exclusive group instead of hardcoding it (Pavel)

Peter Krempa (3):
  scripts: render_block_graph: Fix invalid escape sequence warning with
    python 3.12
  scripts: render_block_graph: Implement proper argument parser
  scripts: render_block_graph: Avoid backtrace on error from virsh

 scripts/render_block_graph.py | 80 +++++++++++++++++++++++++----------
 1 file changed, 57 insertions(+), 23 deletions(-)

-- 
2.48.1

Re: [PATCH v2 0/3] scripts: render_block_graph: Fix with new python and improve argument parsing

Posted by Pavel Hrdina 1 week, 6 days ago

On Wed, Mar 19, 2025 at 05:28:06PM +0100, Peter Krempa wrote:
> New python doesn't like '\l' escapes from the 'dot' language.
> 
> While at it improve usability of the script by employing proper argument
> parsing.
> 
> v2:
>  - use mutually exclusive group instead of hardcoding it (Pavel)
> 
> Peter Krempa (3):
>   scripts: render_block_graph: Fix invalid escape sequence warning with
>     python 3.12
>   scripts: render_block_graph: Implement proper argument parser
>   scripts: render_block_graph: Avoid backtrace on error from virsh

Reviewed-by: Pavel Hrdina <phrdina@redhat.com>