search

p4lang / behavioral-model

The reference P4 software switch
Apache License 2.0
532 stars 327 forks source link

Add enable-debugger option in 1sw_demo.py. #1174

Closed r12f closed 1 year ago

r12f commented 1 year ago

This change adds a new command line option --enable-debugger to enable debugger support in 1sw_demo.py. Once this option is specified, 1sw_demo.py will pass --debugger option to the behavioral exe when launching it. And this will allow us to connect the debugger with pydbg.

One thing to notice is - in order to use this option, we need to use the debugger enabled behavioral exe. Feel free to check the behavioral model readme to see how to compile with debugger support enabled.

More discussions can be found with issue #1173.

r12f commented 1 year ago

ouch! definitely no. let me check what might be happening. it could be the repo setup issue.

r12f commented 1 year ago

thanks a lot for spotting this!

r12f commented 1 year ago

got the nnpy change reverted and pushed a new iteration.

r12f commented 1 year ago

btw, just in case - since this is my first PR, it looks like the workflow will need approval to run. i wonder if anyone could help with it : D.

r12f commented 1 year ago

thanks a lot, Antonin! btw, I am seeing the PR is blocked because "The base branch restricts merging to authorized users.". is there anything else I need to do?