Skip to content

Conversation

@rtakacs
Copy link
Contributor

@rtakacs rtakacs commented Feb 6, 2017

According to #602 I've created a patch to update JerryScript.

@rtakacs rtakacs force-pushed the update_jerry branch 2 times, most recently from 00e3823 to 7a8aeaf Compare February 7, 2017 15:15
@LaszloLango
Copy link
Contributor

@rtakacs after (jerryscript-project/jerryscript@638b753) it should work. Please update to the latest JerryScript.

Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@glistening glistening left a comment

Choose a reason for hiding this comment

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

LGTM

@zherczeg
Copy link
Member

Please wait for #1574 in jerryscript before landing this.

IoT.js-DCO-1.0-Signed-off-by: Roland Takacs rtakacs.u-szeged@partner.samsung.com
@rtakacs
Copy link
Contributor Author

rtakacs commented Feb 10, 2017

@zherczeg : I've updated jerry to 32674ff [Documentation update (#1573)]

@LaszloLango
Copy link
Contributor

Let's suspend this update until we investigate it's effects on memory consumption.

@LaszloLango
Copy link
Contributor

False alarm. Tested the PR and there are no issues.

@LaszloLango LaszloLango reopened this Feb 10, 2017
@LaszloLango LaszloLango merged commit 9c707b1 into jerryscript-project:master Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants