New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed debugger build on TizenRT. #1789
Fixed debugger build on TizenRT. #1789
Conversation
jerryscript-project/jerryscript#2586 is also needed. |
e5cd14c
to
04ac36d
Compare
The PR is ready for review, tested locally. Builds successfully when jerryscript-project/jerryscript#2586 is applied. I opened a PR (#1791) for the necessary JerryScript update. |
May I suggest you to rebase on master and push again |
TizenRT should not build the default port implementation of the JerryScript engine. TizenRT port defines its own port implementation in the 'iotjs_main_tizenrt.c' file. Added missing 'jerry_port_sleep' implementation if JerryScript debugger is enabled. IoT.js-DCO-1.0-Signed-off-by: László Langó llango.u-szeged@partner.samsung.com
04ac36d
to
06c9006
Compare
@rzr of course, the patch is rebased. |
ok https://travis-ci.org/Samsung/iotjs/jobs/452746615 is Verified for tizenrt (using CI's defconfig) |
@rzr thank your for checking it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
TizenRT should not build the default port implementation of the
JerryScript engine. TizenRT port defines its own port implementation
in the 'iotjs_main_tizenrt.c' file. Added missing 'jerry_port_sleep'
implementation if JerryScript debugger is enabled.
IoT.js-DCO-1.0-Signed-off-by: László Langó llango.u-szeged@partner.samsung.com