-
Notifications
You must be signed in to change notification settings - Fork 683
Added target to compile RIOT-OS against jerryscript #1130
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
Conversation
LGTM |
/* Suppress compiler errors */ | ||
(void)argc; | ||
(void)argv; | ||
const jerry_api_char_t script[] ="print ('Hello, World!');"; |
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.
space after =
space after (void)
708e505
to
c0da053
Compare
I've updated the patch, based on your comments. |
|
||
/* start the shell */ | ||
char line_buf[SHELL_DEFAULT_BUFSIZE]; | ||
shell_run(shell_commands, line_buf, SHELL_DEFAULT_BUFSIZE); |
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.
Space after shell_run
c0da053
to
3218b26
Compare
Thanks, I've updated these. |
LGTM |
This patch contains an example in which you can run a small js on RIOT-OS with STM32F4-Discovery. Check the README.md for a more detailed explanation of how to compile and run it. JerryScript-DCO-1.0-Signed-off-by: Robert Sipka rsipka.uszeged@partner.samsung.com
3218b26
to
0dd1082
Compare
Thanks, I've rebased it with master. |
This patch contains an example in which you can run a small js on RIOT-OS with STM32F4-Discovery.
Check the README.md for a more detailed explanation of how to compile and run it.
JerryScript-DCO-1.0-Signed-off-by: Robert Sipka rsipka.uszeged@partner.samsung.com