Skip to content

Conversation

rivos-eblot
Copy link

No description provided.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
Copy link

@AlexJones0 AlexJones0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks. I've left a couple of minor comments/suggestions.

Using an existing Python module name is a very bad idea as it ends up
conflicting when Python interpreter looks up for modules.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
…ype hinting

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
… directory

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
… failure

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
QEMU executer should use QEMU wrapper.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
Some fail test may report no execution time.

Signed-off-by: Emmanuel Blot <eblot@rivosinc.com>
@rivos-eblot rivos-eblot merged commit 8538b81 into lowRISC:ot-9.2.0 Sep 26, 2025
8 checks passed
@rivos-eblot rivos-eblot deleted the dev/ebl/py_tools branch September 26, 2025 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants