intel / zephyr.js

JavaScript* Runtime for Zephyr* OS
Other
180 stars 64 forks source link

[jerryscript-debugger] Fail to resume/step into/step over when debugging with /bin/jerry built in JerryScript #1877

Open qiaojingx opened 6 years ago

qiaojingx commented 6 years ago

Description

When using jerryscript-debugger.sh to debug with /bin/jerry executable inside JerryScript. Breakpoint can be added and removed. But Resume/Step into/Step over/Step out/Step doesn't work. It wil cause "unhandled protocol message type: xx". After clicking once, the icon becomes disabled.

Test Code

JerryScript Debugger & Chrome DevTools Proxy; zephyr.js/samples/Buffer.js;

Steps to Reproduction

  1. $ cd jerryscript
  2. $ python tools/build.py --jerry-debugger=on --jerry-libc=off
  3. $ ./build/bin/jerry --start-debug-server --log-level 2 $ZJS_BASE/samples/Buffer.js
  4. Have Chrome running and visit the URL chrome://inspect, and click "Open dedicated DevTools for Node."
  5. $ cd jerryscript-debugger-ts
  6. $ yarn install
  7. $ ./jerry-debugger.sh
  8. Click Resume/Step into/Step over/Step out/Step

Actual Result

After step7: Error message "incorrect target debugger version detected: 3 expected 2 is print after connection setup. selection_004

After step 8: Step into/Step over/Step out/Step icon become disabled. Error message "unhandled protocol message type: xx" is print in terminal. The icons are disabled: selection_005 Error message in terminal: selection_003

Expected Result

No error occurs. Able to use resume/Step into/Step over/Step out/Step to debug.

Test Builds

Project Commit Id Target Device Test Date Result
jerryscript d672d1e Linux Apr. 27, 2018 Fail
jerryscript-debugger-ts ed66d7d Linux Apr. 27, 2018 Fail

Additional Information

grgustaf commented 6 years ago

OK, sorry yeah if you are able to characterize a bug like this without reference to ZJS, e.g. using bin/jerry, then it would be best to report directly on the jerryscript-debugger-ts project.

qiaojingx commented 6 years ago

We have created a issue in jerryscript-debugger-ts/#12 for using bin/jerry to debug JS file in JerryScript. The error message is also print when connecting and debugging through Resume/Step into/Step over/Step out/Step. The different from issue #1877 is Resume/Step into/Step over/Step out/Step not disabled after one click.