Skip to content

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

Merged
merged 1 commit into from
Jun 9, 2016

Conversation

robertsipka
Copy link
Contributor

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

@LaszloLango LaszloLango added development Feature implementation jerry-port Related to the port API or the default port implementation labels Jun 9, 2016
@LaszloLango
Copy link
Contributor

LGTM

/* Suppress compiler errors */
(void)argc;
(void)argv;
const jerry_api_char_t script[] ="print ('Hello, World!');";
Copy link
Member

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)

@robertsipka
Copy link
Contributor Author

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);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space after shell_run

@robertsipka
Copy link
Contributor Author

Thanks, I've updated these.

@zherczeg
Copy link
Member

zherczeg commented Jun 9, 2016

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
@robertsipka
Copy link
Contributor Author

Thanks, I've rebased it with master.

@bzsolt bzsolt merged commit 0dd1082 into jerryscript-project:master Jun 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Feature implementation jerry-port Related to the port API or the default port implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants