Skip to content

Commit

Permalink
tools: Add various fixes for idf.py gdbgui
Browse files Browse the repository at this point in the history
Closes #5968
  • Loading branch information
dobairoland committed Apr 27, 2021
1 parent 3ed5d97 commit 0102cbf
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
3 changes: 2 additions & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ gdbgui==0.13.2.0
# Windows is not supported since 0.14.0.0. See https://github.com/cs01/gdbgui/issues/348
pygdbmi<=0.9.0.2
# The pygdbmi required max version 0.9.0.2 since 0.9.0.3 is not compatible with latest gdbgui (>=0.13.2.0)
python-socketio<5; python_version<="2.7"
# A compatible Socket.IO should be used. See https://github.com/miguelgrinberg/python-socketio/issues/578
python-socketio<5

kconfiglib==13.7.1

Expand Down
7 changes: 6 additions & 1 deletion tools/idf_py_actions/debug_ext.py
Original file line number Diff line number Diff line change
Expand Up @@ -203,8 +203,13 @@ def gdbui(action, ctx, args, gdbgui_port, gdbinit, require_openocd):
args += ['--port', gdbgui_port]
gdbgui_out_name = os.path.join(local_dir, GDBGUI_OUT_FILE)
gdbgui_out = open(gdbgui_out_name, 'a+')
env = os.environ.copy()
# The only known solution for https://github.com/cs01/gdbgui/issues/359 is to set the following environment
# variable. The greenlet package cannot be downgraded for compatibility with other requirements (gdbgui,
# pygdbmi).
env['PURE_PYTHON'] = '1'
try:
process = subprocess.Popen(args, stdout=gdbgui_out, stderr=subprocess.STDOUT, bufsize=1)
process = subprocess.Popen(args, stdout=gdbgui_out, stderr=subprocess.STDOUT, bufsize=1, env=env)
except Exception as e:
print(e)
raise FatalError('Error starting gdbgui. Please make sure gdbgui can be started', ctx)
Expand Down

0 comments on commit 0102cbf

Please sign in to comment.